Skip to content

LF-5428 Add GrownBy to Market Directory Tables - #4291

Draft
kathyavini wants to merge 2 commits into
integrationfrom
LF-5428-add-grown-by-integration-to-prod-migration
Draft

LF-5428 Add GrownBy to Market Directory Tables#4291
kathyavini wants to merge 2 commits into
integrationfrom
LF-5428-add-grown-by-integration-to-prod-migration

Conversation

@kathyavini

@kathyavini kathyavini commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

Drafted as this is not intended for production yet. I'm opening the PR in draft just to have a record of what has to be done when we're ready to integrate. Now that I think about it, the ticket title was a little stupid because obviously it's not until we merge to main that this goes to production... will rename now 😅

Description

With the frontend already merged (#4290), adding GrownBy to the market_directory_partner and market_directory_partner_country returns it to the specified farms (here, US and Canadian) and shows the partner tile.

This migration can be run locally to test, and it should be safe to run on beta where all the values are already supplied.

Note, as as written in the migration comments, the market_directory_partner_auth has to be added before the integration (webhook + keycloak) will work; it is not included in the migration becuase the values differ per environment.

Test file change

The test fix addresses an error I don't know if I've ever seen on GitHub actions; it's one of those issues that is only present if another test hasn't already dropped the table and so the migrated data still stands. You can see it by running (only) marketDirectoryPartner.test.ts after a fresh migration of the test db, without this fix. You can see it on integration too if you do that, with OFN Canada being the trigger.

The fix makes sure the partners are only compared against the records that are added by the test, so it doesn't matter if the migrated records are still present or not.

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

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Passes test case
  • UI components visually reviewed on desktop view
  • UI components visually reviewed on mobile view
  • Other (please explain)

Ran the migration to generate the tile locally, although the webhook / integration is really only testable on beta.

Checklist:

  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • The precommit and linting ran successfully
  • I have added or updated language tags for text that's part of the UI
  • I have ordered translation keys alphabetically (optional: run pnpm i18n to help with this)
  • I have added the GNU General Public License to all new files

@kathyavini kathyavini self-assigned this Aug 1, 2026
@kathyavini
kathyavini requested a review from a team as a code owner August 1, 2026 00:07
@kathyavini kathyavini added the enhancement New feature or request label Aug 1, 2026
@kathyavini
kathyavini requested a review from a team as a code owner August 1, 2026 00:07
@kathyavini
kathyavini requested review from SayakaOno and removed request for a team August 1, 2026 00:07
@kathyavini
kathyavini marked this pull request as draft August 1, 2026 00:09
@kathyavini kathyavini changed the title LF-5428 Add GrownBy Integration to Prod (Migration) LF-5428 Add GrownBy to Market Directory Tables Aug 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant