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

ai_reports/serializer: use get() for label #60

Merged
merged 1 commit into from
Sep 19, 2024
Merged

ai_reports/serializer: use get() for label #60

merged 1 commit into from
Sep 19, 2024

Conversation

goapunk
Copy link

@goapunk goapunk commented Sep 19, 2024 •
edited
Loading

ai_reports/serializer: use get() for label to prevent errors for unknown keys

Tasks

  • PR name contains story or task reference
  • Documentation (docs and inline)
  • Tests (including n+1 and django_assert_num_queries where applicable)
  • Changelog

@goapunk goapunk changed the title ai_reports/serializer: use get() for label to prevent errors for unkn… ai_reports/serializer: use get() for label Sep 19, 2024
@goapunk goapunk requested review from m4ra and hom3mad3 September 19, 2024 12:53
@goapunk
Copy link
Author

goapunk commented Sep 19, 2024

we have some unknown labels on our dev server, this should fix it

@m4ra m4ra merged commit ca3aac5 into main Sep 19, 2024
3 checks passed
@m4ra m4ra deleted the jd-2024-09-use-get-xai-label branch September 19, 2024 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m4ra m4ra m4ra approved these changes

@hom3mad3 hom3mad3 Awaiting requested review from hom3mad3

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@goapunk @m4ra