Skip to content

Conversation

@Luke-Oldenburg
Copy link
Contributor

Summary of the problem

Describe your changes

@Luke-Oldenburg Luke-Oldenburg requested review from a team as code owners July 1, 2025 15:57
Copy link
Member

@garyhtou garyhtou left a comment

Choose a reason for hiding this comment

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

Instead of doing an after_update which creates a CardGrant for all organizations, let's always show the Card Grant settings tab if the event plan allows for it. On that tab, if the card grant setting doesn't exist, then show an "Enable card grants" button.

@garyhtou
Copy link
Member

garyhtou commented Jul 1, 2025

The downside of after_update is that we'd have a bunch of unused card grant settings. When we need to change the default settings, we'd then need to determine which are actually used vs unused, communicate breaking changes, and roll out changes to these records.

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.

4 participants