Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kowalski filters: support new keywords #549

Merged
merged 1 commit into from
May 25, 2024
Merged

Conversation

Theodlz
Copy link
Collaborator

@Theodlz Theodlz commented May 25, 2024

Add support for:

  • autosave.saver_id, allowing Kowalski to use a different user account to autosave candidates as sources to a group, useful to not have everything posted by the kowalski-bot account both for book-keeping and to allow TNS autoreports from properly configured bot accounts.
  • auto_followup.priority_order, so Kowalski knows if the instrument API associated to the allocation selected has descending or ascending priority values.
  • auto_followup.implements_update, so Kowalski knows if the priority of an existing request can be updated or not.

…iority_order and auto_followup.implements_update
@Theodlz Theodlz requested a review from mcoughlin May 25, 2024 17:15
Copy link
Contributor

@mcoughlin mcoughlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Theodlz Theodlz merged commit 76842ec into main May 25, 2024
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants