Pages Editor: remove "Workflows belongs to Project" checks#7344
Open
shaunanoordin wants to merge 1 commit intomainfrom
Open
Pages Editor: remove "Workflows belongs to Project" checks#7344shaunanoordin wants to merge 1 commit intomainfrom
shaunanoordin wants to merge 1 commit intomainfrom
Conversation
Member
Author
|
@zooniverse/frontend please take a look at my PR thanks! (I'm actually testing the @ system) |
45555d5 to
006b32b
Compare
006b32b to
68ae71e
Compare
…d as part of project
68ae71e to
336cf7c
Compare
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
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Overview
Fixes #7343
Affects: Pages Editor (aka Workflow Editor)
This PR fixes an issue where newly-created workflows aren't correctly identified as being part of the project. This means the Pages Editor no longer checks if the workflow belongs to a project before allowing users to edit the workflow.
The "workflow belongs to project" check was initially added as a safety against a niche scenario, but it seems to be causing more problems than it's solving.
Testing
Testing is a bit difficult, so don't worry about it too much since this only affects the new Pages Editor (aka Workflow Editor) and NOT the standard PFE Project Builder page for editing workflows.
But if you really want to...
Basic test for Pages Editor:
Specific tests for PR:
The problem with running these tests is that I had to set up two different projects with two different user accounts, which was a pain to do properly. 😬 You'd either need to trust me that I set up the workflows and zootester1/2 permissions properly, or you'll need to replicate the permissions etc correctly. Trying to test any of the above with our standard admin mode enabled will just bypass a bunch of the permission checks anyway.
Status
Low priority, ready for review.