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

Disabled Add More + button in submission #2535

Closed
MarieVerdonck opened this issue Oct 4, 2023 · 3 comments
Closed

Disabled Add More + button in submission #2535

MarieVerdonck opened this issue Oct 4, 2023 · 3 comments
Assignees
Milestone

Comments

@MarieVerdonck
Copy link
Contributor

MarieVerdonck commented Oct 4, 2023

Describe the bug
What is the function of the (disabled) [+Add More] button in top right corner of submission? Based on the code this button would have functionality if there are 'disabled' sections, how are they defined?
Screenshot 2023-10-04 at 17 21 54

Possibly:
In item-submission.xml steps can have a visibility part in the scope, with options: HIDDEN("hidden"), READONLY("read-only"), EDITABLE("editable") => collection selection is supposedly 'hidden' though in submission, & is not part of these "disabled" sections
https://demo.dspace.org/server/#https://demo.dspace.org/server/api/config/submissionsections => visibility for collection step not set.

Disabled sections are sections with mandatory=false for the step-definition in item-submission.xml

In any case if there are no such 'disabled' sections should the [+Add More] button not be hidden instead of just disabled

To Reproduce
Steps to reproduce the behavior:

  1. Go to submission in regular submission form, eg https://demo.dspace.org/workspaceitems/33406/edit
  2. There is a disabled [+Add More] button in top right corner of submission causing confusion

Expected behavior
Button hidden if no 'disabled' sections
Unclear what disabled sections are / how button can be used if there are such sections

Related work
Link to any related tickets or PRs here.
Disabled instead of hide in case of no disabled section done in c73b4c7
Part of:
https://github.com/DSpace/dspace-angular/pull/279/commits
https://github.com/DSpace/dspace-angular/pull/381/commits

@MarieVerdonck MarieVerdonck added bug component: submission needs triage New issue needs triage and/or scheduling labels Oct 4, 2023
@tdonohue tdonohue removed the needs triage New issue needs triage and/or scheduling label Oct 4, 2023
@hutattedonmyarm
Copy link
Contributor

Disabled sections are sections with mandatory=false

They will not be displayed by default in the submission process, but can be added with this button

@MarieVerdonck
Copy link
Contributor Author

@hutattedonmyarm
Thanks for this info! Updated this issue & the related PR's instructions for reviewers with this.

Verified setting mandatory="false" in item-submission.xml: <step-definition id="traditionalpagetwo" mandatory="false"> on the default second describe step does indeed make this add button show again after changes of PR, and is not disabled but a dropdown allowing you to add this non-mandatory step.

@alanorth
Copy link
Contributor

Closed with #2536 for dspace-7_x and #2544 for main.

@tdonohue tdonohue added this to the 7.6.1 milestone Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

4 participants