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

Workflow repositories step doesn't honor selected status of repositories #948

Open
jabrah opened this issue May 31, 2019 · 2 comments
Open
Assignees

Comments

@jabrah
Copy link
Contributor

jabrah commented May 31, 2019

The selected status of an optional repository is resent when a user navigates back and forth within the submission workflow.

  • Progress through submission workflow
  • Deselect J10P repo
  • Click Next to persist repositories on submission
  • Click Back to reload the Repositories step
  • J10P is selected again

This may be tricky to handle for draft submissions because it may not have the all of the information present within the workflow. Some values are held in non-persistent variables, such as repository._selected

@jabrah jabrah added the bug label May 31, 2019
@htpvu htpvu added the blocker critical task - needs to be worked on before launch label Jun 18, 2019
@jabrah
Copy link
Contributor Author

jabrah commented Jul 2, 2019

There are steps we can take to mitigate this bug, but I don't think it can be completely solved at this point.

For the scenario above, we could save the selection status in the in workflow.js which holds data through the workflow. This should address the repository selection navigating back and forth in the workflow.

However, there is the problem of loading a draft submission and going through the workflow. In this case, I don't think there is anything persisted with the submission that will tell us about the repository selection status with complete accuracy. We could attempt to infer the selection status, given enough information in the draft submission, such as saved repositories, policies, and the journal. This approach seems like it would be error-prone, though.

@markpatton
Copy link
Contributor

This has already been fixed for non-draft submissions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants