feat(projects): add re-certification flow for returned ships#442
Merged
Conversation
Collaborator
|
This looks good. One tiny non blocking thing. the pending review check here runs before the lock so if anyone with iffy wifi double click it. two request can hit the server at the same time. You should move the check inside the lock so it actually guardss! |
Contributor
Author
@dhamariT Good point, this is fixed now :) |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
what's this do?
Once a project was returned with changes requested, there was no way to re-submit it for certification.
So now the "Changes requested" badge on ship cards is now a clickable button that opens a modal displaying the shipwright's proof video, feedback, and a "Request re-certification" button :)
Requesting re-certification resets the ship event's certification_status back to "pending" :)
show it works
ai?
Claude