-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add ability to resubmit boundary #185
Conversation
2a025f2
to
d8fd1e1
Compare
src/app/src/utils.js
Outdated
@@ -131,6 +131,7 @@ export function getBoundaryPermissions({ boundary, user }) { | |||
canWrite: false, |
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.
I'll be updating this function again in the PR for #151 to separate the role and boundary aspects of these permissions. It might not be worth getting into too much detail here.
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.
+1 tested. Nice work including the previous submission in the API and making the choice in the front-end about which annotations to show.
if (canWrite && boundary.status === BOUNDARY_STATUS.NEEDS_REVISIONS) { | ||
label = 'Revise boundary'; | ||
if (canCreateDraft && boundary.status === BOUNDARY_STATUS.NEEDS_REVISIONS) { | ||
label = 'Revise map'; |
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.
Apologies, this should still be Review boundary.
@rajadain I accidentally commited Split boundary and user permission logic to this branch when I meant to commit to add-support-for-approving-boundaries. I was going to cherry-pick it off this branch, but then I decided not to because it does override some of the changes from this PR (and works without the changes from the newer branch). I think it happened while you were reviewing so you might not have seen those changes. Let me know if you want me to move forward with cherry-pick or if you want to review it here. |
Will take a look here |
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.
Tested all the permissions use cases, this is working well. Nice job!
4c866ef
to
41f5056
Compare
Also updates canApprove logic and adds canUnapprove logic
41f5056
to
f33fc7e
Compare
Overview
This PR adds the ability to resubmit boundaries.
Closes #180
Demo
Screen.Recording.2022-11-07.at.11.56.39.AM-1.mov
Notes
Showing annotations on the next draft required returning previous submission info from the boundary details route.
Testing Instructions
Checklist
fixup!
commits have been squashedCHANGELOG.md
updated with summary of features or fixes, following Keep a Changelog guidelinesREADME.md
updated if necessary to reflect the changes