活动限定奖池 #837
Merged
活动限定奖池 #837
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Make the exchange attributes optional Commit migration file
pkuGenuine
reviewed
Aug 30, 2024
Removed the NULL flag because we have a non-null default value.
So if the user didn't take part in the required activity, the pool won't show up in the `showPools` page.
Use class instead of string when specifying foreign key; add None as default for Pool.activity
pkuGenuine
approved these changes
Aug 31, 2024
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
为奖池增加了一个可选的
activity外键，如果非空，会在用户兑换前检查是否已参加这个活动。如果未参加，将拒绝用户的兑换/抽奖/盲盒请求：顺便修了一下 #834 中的一个小问题：把奖池奖品表中的“属性”改为可选，不然所有的记录都必须要设置非空的属性。