-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
Fs 4064 allocation association table #481
Conversation
f86ac22
to
a39c7cf
Compare
a39c7cf
to
53f443d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Looks good. Only thought is.. |
53f443d
to
ef72b12
Compare
ef72b12
to
8a6e591
Compare
(Discussed this in standup and agreed this is the desirable approach. Posting here to keep a record). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Change description
Added allocation association table as a prerequisite for assigning assessors to assessments.
How to test
N/A (manually tests upgrade / downgrade of alembic revision)
Screenshots of UI changes (if applicable)