Skip to content

Fix single badge type selection - #48

Merged
james7132 merged 1 commit into
touhoufest2027from
single-day-selection
Sep 11, 2026
Merged

james7132 merged 1 commit into
touhoufest2027from
single-day-selection

Conversation

@7cff

@7cff 7cff commented Sep 11, 2026

Copy link
Copy Markdown
Member

This is a backport from upstream to make the UI code detect that a day for the single-day badge was actually selected.

Comment thread uber/forms/attendee.py


class BadgeExtras(MagForm):
field_aliases = {'badge_type': ['badge_type_single']}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ [ruff] <RUF012> reported by reviewdog 🐶
Mutable default value for class attribute

Comment thread uber/forms/attendee.py

class BadgeExtras(MagForm):
field_aliases = {'badge_type': ['badge_type_single']}
dynamic_choices_fields = {'shirt': lambda: c.SHIRT_OPTS, 'staff_shirt': lambda: c.STAFF_SHIRT_OPTS}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ [ruff] <RUF012> reported by reviewdog 🐶
Mutable default value for class attribute

@james7132 james7132 added the U-Upstreamed This is a issue or pull request that has already been upstreamed to MAGFest. label Sep 11, 2026
@james7132
james7132 merged commit 6293763 into touhoufest2027 Sep 11, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

U-Upstreamed This is a issue or pull request that has already been upstreamed to MAGFest.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants