Skip to content

Fix: Duplicate funding reference add button - #1278

Merged
McNamara84 merged 1 commit into
mainfrom
fix/issue-1274
Sep 8, 2026
Merged

Fix: Duplicate funding reference add button#1278
McNamara84 merged 1 commit into
mainfrom
fix/issue-1274

Conversation

@McNamara84

Copy link
Copy Markdown
Owner

This pull request improves the user experience for adding funding references in the Data Editor by ensuring that only one clear "Add Funding Reference" action is visible at any time, regardless of whether the list is empty or contains items. It also updates and adds comprehensive tests to verify this behavior, and documents the change in the changelog.

User Interface Improvements:

  • The FundingReferenceField component now displays a single "Add Funding Reference" button when the funding reference list is empty, and maintains a single list-level add button when references exist. This prevents duplicate prompts and clarifies the add action for users.

Testing Enhancements:

  • Added and updated tests in funding-reference-field.test.tsx to ensure that:
    • Only one add action is shown when no references exist.
    • Only one add action is shown when references exist, and clicking it appends a new reference.
    • The UI returns to a single empty-state add action after all references are removed.
    • Tests updated to select the add button more robustly and avoid using array selectors. [1] [2]

Documentation:

  • Added a changelog entry describing the improved add action behavior for funding references.
  • Updated the API changelog test to assert the presence of the new entry.

Show only one "Add Funding Reference" action when the funding section is empty, while keeping the list-level add button for existing entries. The changelog and API/frontend tests were updated to cover the new empty-state and append behavior.
@McNamara84 McNamara84 added this to the v1.0.8 milestone Sep 8, 2026
@McNamara84 McNamara84 self-assigned this Sep 8, 2026
@McNamara84 McNamara84 added the bug Something isn't working label Sep 8, 2026
@McNamara84 McNamara84 linked an issue Sep 8, 2026 that may be closed by this pull request
7 tasks
@codecov

codecov Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@McNamara84
McNamara84 marked this pull request as ready for review September 8, 2026 04:41
@McNamara84
McNamara84 requested a balanced review from Copilot September 8, 2026 04:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The changelog test newly asserts a release date that is still in the future.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Prevents duplicate funding-reference add actions in the Data Editor and adds regression coverage.

Changes:

  • Shows one context-appropriate add button.
  • Tests empty, populated, and post-removal states.
  • Documents the fix in the changelog.
File summaries
File Description
funding-reference-field.tsx Hides the list-level add button when empty.
funding-reference-field.test.tsx Adds regression tests for add-action behavior.
ChangelogApiTest.php Verifies the changelog entry.
changelog.json Documents the UI fix.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@McNamara84
McNamara84 merged commit 4eb2256 into main Sep 8, 2026
30 checks passed
@McNamara84
McNamara84 deleted the fix/issue-1274 branch September 8, 2026 04:48
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

Successfully merging this pull request may close these issues.

Duplicate display of “Add Found References”

2 participants