Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Patch 3.6.5 #3267

Merged
merged 183 commits into from
Jul 25, 2024
Merged

Patch 3.6.5 #3267

merged 183 commits into from
Jul 25, 2024

Conversation

SayakaOno
Copy link
Collaborator

@SayakaOno SayakaOno commented Jun 25, 2024

Description

Branched out from the main branch and mainly cherry-picked merge commits from the integration branch.

Main features included:

  • Fallback screen
  • Button colours update
  • NumberInput component
  • ReactSelect conversion to TypeScript
  • Soil Amendment

I also cherry-picked individual commits where files were touched in different PRs after the merge commit, like NumberInput.
There were many commits made to colors.scss, so I didn't cherry-pick all of them. We could try cherry-picking more commits if we want, or use git checkout integration -- <file> to grab the changes from the integration branch. However, I believe the missing colours in this PR are for animals.

Merge conflicts are expected in these files when merging main into integration, but they should be straightforward to resolve.

Screenshot 2024-06-24 at 5 42 23 PM

Jira link: https://lite-farm.atlassian.net/browse/LF-4231

antsgar and others added 13 commits June 24, 2024 13:07
…een-instead-of-white-screen

LF-3946 Show a fallback screen instead of white screen
And lint the scss file as I think only my editor does so
…rs-and-variants-throughout-app

LF-4178 Update button colours and variants throughout app
…ber-input-with-localization-support

Lf 3934 create separate number input with localization support
* cherry-pick be23eb6, remove change in SexDetailsCountInput
* cherry-pick b469ee9, remove changes in SexDetailsCountInput and SexDetails
* cherry-pick 08d3fc8, remove changes in AddAnimalsFormCard
* cherry-pick 7549d0d, remove changes in AddAnimalsFormCard and AddAnimalsFormCard scss
@SayakaOno SayakaOno changed the base branch from integration to main June 25, 2024 00:19
@SayakaOno SayakaOno changed the base branch from main to integration June 25, 2024 00:42
@SayakaOno SayakaOno changed the base branch from integration to main June 25, 2024 00:42
SayakaOno and others added 8 commits June 25, 2024 14:34
…add-columns-to-product-table-and-create-update-ap-is

LF-4215 Write migration to add columns to product table and create update APIs
…r_product_section_buttons

LF-4217: Create component for product section buttons
* cherry-pick bf21908, remove KPI Dashboard changes
Also lint the mixin file
kathyavini and others added 17 commits July 24, 2024 09:05
…-only-it-might-be-too-much-to-have-name-and-npk-all-in-select-option-and-there-should-probably-be-some-padding-between-the-end-of-product-name-and-n

LF-4342 For the mobile view only it might be too much to have name and npk all in select option and there should probably be some padding between the end of product name and n
…atch/3.6.5_api_job_locales

New Crowdin translations
…nput_field

LF-4315: Magnetic supplier input field
…er-to-dry-or-liquid

patch/3.6.5 update input text based upon farmer feedback
…nitially undefined

This was the issue happening on Duncan and Sayaka's local env but not mine -- mysterious
…e-to-complete-a-task-any-task-for-a-farm-worker-account-without-an-email

LF-4346 The user is not able to complete a task any task for a farm worker account without an email
…atch/3.6.5_webapp_locales

New Crowdin translations
LF-4348 Demo branch -- Pass other_purpose_id as a form field
@kathyavini kathyavini marked this pull request as ready for review July 25, 2024 18:11
@kathyavini kathyavini requested review from a team as code owners July 25, 2024 18:11
@kathyavini kathyavini requested review from Duncan-Brain and kathyavini and removed request for a team July 25, 2024 18:11
@Duncan-Brain
Copy link
Collaborator

WOOO LGTM

@kathyavini kathyavini merged commit e578213 into main Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants