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

When embargo limitation rule is violated, return user to that question in the form, not the top of the page #3623

Open
amyehodge opened this issue Sep 25, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@amyehodge
Copy link
Collaborator

If a user has entered an embargo date that is past the allowed amount of time set for the collection, and then tries to deposit, we run validation that catches this and then the following things should happen:

  1. an error message displays next to the embargo date entry fields in the form
Screenshot 2024-09-25 at 1 03 34 PM
  1. the user is returned to this section of the form to make the correction. Note that if there are also other errors higher up on the page, the user is taken to the error nearest the top of the page.

Currently, the error message is also displaying in a big pink box at the top of the page and the user is being taken to the very top of the page instead of the embargo section. To see the current state:

  1. open this item in stage: https://sul-h2-stage.stanford.edu/works/662
  2. click the pencil icon to edit
  3. scroll to the bottom and click deposit
  4. note that you are taken to the top of the page and that the error is displayed there where we don't want it. this message should be suppressed.
Screenshot 2024-09-25 at 1 03 42 PM 5. scroll down to the embargo section to that the desired error message being properly displayed here
@amyehodge amyehodge added the bug Something isn't working label Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant