Skip to content
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

Disable editing of context and controls for author when the request is under review #538

Merged
merged 5 commits into from
Jul 16, 2024

Conversation

rebkwok
Copy link
Contributor

@rebkwok rebkwok commented Jul 16, 2024

Disable editing of context/controls for author when the request is under review
Fixes #537

Hide comments unless allowed to comment
Fixes #539
Commenting is only allowed if a request is
- not final
- is under review, and the user is an output-checker
- is in draft, and the user has permission to accesss the workspace

@rebkwok rebkwok marked this pull request as draft July 16, 2024 08:34
@rebkwok rebkwok force-pushed the disable-context-controls branch 3 times, most recently from 296ec98 to 9a3718e Compare July 16, 2024 09:25
@rebkwok rebkwok marked this pull request as ready for review July 16, 2024 09:25
Copy link
Member

@bloodearnest bloodearnest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Minor nit: is_in_draft doesn't quite read right for me when the the request is RETURNED, but I don't have any better suggestions, really

@rebkwok
Copy link
Contributor Author

rebkwok commented Jul 16, 2024

Yeah, I spent a while trying to come up with a decent name for it. in_progress isn't specific enough, is_editing and is_under_edit sounded weird, and didn't really feel like they covered the PENDING stage. Is is_editing better that is_in_draft?

rebkwok added 4 commits July 16, 2024 14:17
Commenting is only allowed if a request is
- not final
- is under review, and the user is an output-checker
- is in draft, and the user has permission to accesss the workspace

Note: in draft status, project collaborators are allowed to answer output checkers'
questions, not just the author
@rebkwok rebkwok force-pushed the disable-context-controls branch from 9a3718e to 92caf75 Compare July 16, 2024 13:18
@bloodearnest
Copy link
Member

Yeah, I spent a while trying to come up with a decent name for it. in_progress isn't specific enough, is_editing and is_under_edit sounded weird, and didn't really feel like they covered the PENDING stage. Is is_editing better that is_in_draft?

yeah, I think its a bit better if you feel like making the change.

@rebkwok rebkwok merged commit 012d7e0 into main Jul 16, 2024
10 checks passed
@rebkwok rebkwok deleted the disable-context-controls branch July 16, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants