Skip to content

Commit

Permalink
Merge pull request #266 from eurofurence/issue-265-bug-required-addon…
Browse files Browse the repository at this point in the history
…s-handling

correctly handle required add-ons
  • Loading branch information
Jumpy-Squirrel authored Feb 16, 2025
2 parents f7fb372 + 8cc6d39 commit 9f59d6e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ const TicketLevelAddon = ({ addon, formContext }: TicketLevelAddonProps) => {
if (resetOnLevelChange && name === 'level' && type === 'change') {
const levelValue = value.level as Exclude<typeof value.level, undefined>

setValue(`addons.${addon.id}.selected`, isIncluded(levelValue) && !isUnavailable(levelValue) || addon.default)
setValue(`addons.${addon.id}.selected`, (isIncluded(levelValue) || isRequired(levelValue)) && !isUnavailable(levelValue) || addon.default)
}

if (name) {
Expand Down Expand Up @@ -67,7 +67,7 @@ const TicketLevelAddon = ({ addon, formContext }: TicketLevelAddonProps) => {
label={addon.id}
description={addon.id}
price={isIncluded(level) ? 0 : addon.price}
disabled={isIncluded(level) || isRequired(level)}
disabled={isIncluded(level) || isRequired(level) || addon.id === 'stage-pass'}
{...register(`addons.${addon.id}.selected`)}
>
{Object.entries(addon.options).map(([id, option]) =>
Expand Down
3 changes: 3 additions & 0 deletions src/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -186,6 +186,9 @@ const configEf = {
type: ['day'],
},
hidden: false,
resetOn: {
levelChange: true,
},
},
'tshirt': {
price: 20,
Expand Down

0 comments on commit 9f59d6e

Please sign in to comment.