Skip to content

Double submit in issue tracker#7095

Merged
labkey-klum merged 1 commit intorelease25.7-SNAPSHOTfrom
25.7_fb_issue_53907
Oct 8, 2025
Merged

Double submit in issue tracker#7095
labkey-klum merged 1 commit intorelease25.7-SNAPSHOTfrom
25.7_fb_issue_53907

Conversation

@labkey-klum
Copy link
Contributor

@labkey-klum labkey-klum commented Oct 2, 2025

Rationale

tracking issue

  • Disable all save buttons on form submit
  • Fix event handler registration, match fields by name case insensitive
  • Re-organize javascript code

Tasks 📍

  • Manual Testing/ Verify Fix @labkey-martyp
    • verify all save buttons are disabled
    • test tab order
    • test dirty state on all form fields

refactor javascript code
fix form element event handlers
Copy link
Contributor

@labkey-martyp labkey-martyp left a comment

Choose a reason for hiding this comment

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

Looks good. Code reviewed and manually tested.

@labkey-klum labkey-klum merged commit 235f768 into release25.7-SNAPSHOT Oct 8, 2025
11 checks passed
@labkey-klum labkey-klum deleted the 25.7_fb_issue_53907 branch October 8, 2025 16:39
@labkey-martyp labkey-martyp restored the 25.7_fb_issue_53907 branch October 14, 2025 23:26
@labkey-martyp labkey-martyp deleted the 25.7_fb_issue_53907 branch October 14, 2025 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants