fix(reviewer): claiming/unclaiming for ship review#305
Merged
Conversation
Member
|
@ObayM this doesn't feel very railsy. i'd encourage asking your claude "how to make claim / unclaim pretend to be a CRUD action" |
Contributor
Author
|
@maxwofford Thanks for the suggestion :) I was initially under the impression that custom actions (post :claim) was the correct pattern in the codebase, so I went with it, but it's now much cleaner :) |
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
3 participants
what's this do?
Adds a way to unclaim your review, fixed the claim functionality as it was resetting the TTL on every page load which was very buggy and might be confusing to shipwrights, it's now a hard 30-minute lock from when you claimed it :)
Note: The UI is will have a rework. It is currently unpolished as it is not a priority for noww
show it works
ai?
Claude