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.

Sign up for GitHub

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

Jump to bottom

Create proper enums for Job statuses #370

Closed
msm-cert opened this issue Feb 5, 2024 · 0 comments · Fixed by #412
Closed

Create proper enums for Job statuses #370

msm-cert opened this issue Feb 5, 2024 · 0 comments · Fixed by #412
Assignees
@michalkrzem
Labels
next-sprint Scheduled for work (CERT.PL internal) zone:backend Backend oriented tasks
Milestone
v1.5.0

Comments

@msm-cert
Copy link
Member

msm-cert commented Feb 5, 2024

Related to #368. Mentioned https://github.com/CERT-Polska/mquery/pull/368/files#r1478115887

Instead of saving a status string like "new" to a database, we should have a proper enum value there.
@msm-cert msm-cert mentioned this issue Feb 5, 2024
Merged
3 tasks
@msm-cert msm-cert added next-sprint Scheduled for work (CERT.PL internal) zone:backend Backend oriented tasks labels Sep 13, 2024
@msm-cert msm-cert mentioned this issue Sep 16, 2024
Open
@msm-cert msm-cert added this to the v1.5.0 milestone Sep 29, 2024
@michalkrzem michalkrzem mentioned this issue Oct 1, 2024
Merged
4 tasks
@michalkrzem michalkrzem linked a pull request Oct 1, 2024 that will close this issue
Create proper enums for job statuses #412
Merged
4 tasks
@msm-cert msm-cert modified the milestones: v1.5.0, Sprint 1 Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michalkrzem michalkrzem

Labels
next-sprint Scheduled for work (CERT.PL internal) zone:backend Backend oriented tasks
Projects
None yet
Milestone
v1.5.0
Development

Successfully merging a pull request may close this issue.

Create proper enums for job statuses CERT-Polska/mquery
2 participants
@michalkrzem @msm-cert