Skip to content

#3888 Create New Material Button for RR Form#3976

Open
rbessin wants to merge 2 commits intofeature/bom-improvementsfrom
#3888-Create-New-Material-Button-RR-Form
Open

#3888 Create New Material Button for RR Form#3976
rbessin wants to merge 2 commits intofeature/bom-improvementsfrom
#3888-Create-New-Material-Button-RR-Form

Conversation

@rbessin
Copy link
Contributor

@rbessin rbessin commented Feb 16, 2026

Changes

ReimbursementRequestForm.tsx

  • Now passes full Project objects to child components
  • Creates formatted autocomplete options for project dropdown

ReimbursementRequestFormView.tsx

  • Updated to receive and pass through full Project data
  • Separated autocomplete options as distinct prop

ReimbursementProductTable.tsx

  • Added "Create New Material" button next to product name field (matching the vendor creation pattern)
  • Implemented material creation modal integration with dynamic assembly fetching
  • Button only visible for Project-based products (hidden for "Other Categories")
  • Auto-populates product name field with created material name

CreateMaterialModal.tsx

  • Added callback to return created material name to parent component

Screenshots

image image

Checklist

It can be helpful to check the Checks and Files changed tabs.
Please review the contributor guide and reach out to your Tech Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.

  • All commits are tagged with the ticket number
  • No linting errors / newline at end of file warnings
  • All code follows repository-configured prettier formatting
  • No merge conflicts
  • All checks passing
  • Screenshots of UI changes (see Screenshots section)
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • No yarn.lock changes (unless dependencies have changed)
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (fill in the closes line below)

Closes # (issue #3888 )

@rbessin rbessin requested a review from wavehassman February 16, 2026 23:17
@rbessin rbessin self-assigned this Feb 16, 2026
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.

1 participant