Skip to content

Scroll to first missing metadata error - #12634

Merged
jp-tosca merged 1 commit into
IQSS:developfrom
GlobalDataverseCommunityConsortium:scrollToMissingReqMD
Sep 1, 2026
Merged

Scroll to first missing metadata error#12634
jp-tosca merged 1 commit into
IQSS:developfrom
GlobalDataverseCommunityConsortium:scrollToMissingReqMD

Conversation

@qqmyers

@qqmyers qqmyers commented Aug 21, 2026

Copy link
Copy Markdown
Member

What this PR does / why we need it: This PR adds a simple update to scroll to the first missing metadata error when a required field is not filled in.

Which issue(s) this PR closes:

Special notes for your reviewer: Should handle first view of the metadata edit page as well as when 'save' is hit.

Suggestions on how to test this: - Leave required metadata empty and hit save, e.g. create a dataset w/o entering anything (so missing title), or delete the contact email in an existing dataset. Can also create an incomplete dataset via the api (when enabled) and go to the dataset page and edit.

Does this PR introduce a user interface change? If mockups are available, please link/include them here: just scrolls, changes top message to Required fields were missed or there was a validation error. See details below. since it now auto scrolls.

Is there a release notes update needed for this change?:

Additional documentation:

@pdurbin pdurbin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved during Triage Tuesday. Fixes an old JSF issue Gary reported long ago.

@github-project-automation github-project-automation Bot moved this from Ready for Triage to Ready for QA ⏩ in IQSS Dataverse Project Aug 25, 2026
@cmbz cmbz added FY27 Sprint 4 FY27 Sprint 4 (2026-08-12 - 2026-08-26) FY27 Sprint 5 FY27 Sprint 5 (2026-08-26 - 2026-09-09) labels Aug 26, 2026
@jp-tosca jp-tosca self-assigned this Aug 26, 2026
@pdurbin pdurbin moved this from Ready for QA ⏩ to QA ✅ in IQSS Dataverse Project Aug 27, 2026
@jp-tosca jp-tosca moved this from QA ✅ to Merged 🚀 in IQSS Dataverse Project Sep 1, 2026
@jp-tosca
jp-tosca merged commit 9f817ab into IQSS:develop Sep 1, 2026
11 checks passed
@jp-tosca jp-tosca removed their assignment Sep 1, 2026
@pdurbin pdurbin moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project Sep 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FY27 Sprint 4 FY27 Sprint 4 (2026-08-12 - 2026-08-26) FY27 Sprint 5 FY27 Sprint 5 (2026-08-26 - 2026-09-09) Size: 3 A percentage of a sprint. 2.1 hours.

Projects

Status: Done 🧹

4 participants