Skip to content

feat: [NR-NMP-462] Update berry plants per acre / distance between rows when updating the other - #487

Merged
wilwong89 merged 5 commits into
mainfrom
feature/berry-fill
Oct 28, 2025
Merged

feat: [NR-NMP-462] Update berry plants per acre / distance between rows when updating the other#487
wilwong89 merged 5 commits into
mainfrom
feature/berry-fill

Conversation

@wilwong89

@wilwong89 wilwong89 commented Sep 22, 2025

Copy link
Copy Markdown
Contributor

Pull Request Standards

  • The title of the PR is accurate
  • The title includes the type of change [HOTFIX, FEATURE, etc]
  • The PR title includes the ticket number in format of [NMP-###]
  • Documentation is updated to reflect change

Description

This PR includes the following proposed change(s):

  • Update form handling methods in CropsModal.ts to autoselect berry plants per acre / distance between rows when selecting the other option.

Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

Comment thread frontend/src/views/Crops/CropsModal.tsx Outdated
@wilwong89

Copy link
Copy Markdown
Contributor Author

Ticket complete as of this comment. New implementation with merged tables and removal of old tables. Frontend work applied. Needs PR review when work resumes.

@raarielgrace raarielgrace 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.

Works on the website, approving with the assumption that you'll make the minor changes.

Comment thread frontend/src/views/Crops/CropsModal.tsx Outdated
Comment thread frontend/src/views/Crops/CropsModal.tsx Outdated
@wilwong89
wilwong89 merged commit 1e414c0 into main Oct 28, 2025
21 checks passed
@wilwong89
wilwong89 deleted the feature/berry-fill branch October 28, 2025 23:15
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.

auto fill Distance fields based on number of plants per acre

3 participants