Skip to content

LF-5380: Add certifications section to market directory#4261

Merged
SayakaOno merged 5 commits into
integrationfrom
LF-5380/Add_certifications_section_to_market_directory
Jul 16, 2026
Merged

LF-5380: Add certifications section to market directory#4261
SayakaOno merged 5 commits into
integrationfrom
LF-5380/Add_certifications_section_to_market_directory

Conversation

@SayakaOno

@SayakaOno SayakaOno commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Description

Adds a read-only "Certifications" card to the Market Directory profile page (/farm_settings/market_directory.

  • MarketDirectoryCertifications (new component)
    • Filters certifications to only those that are actually publishable (active and not expired; Pursuing and Expired are excluded entirely).
    • Shows CertificationsEmptyState when there are none, or a list of CertificationCards (edit icon only, no delete) when there are — clicking edit navigates to /certifications/:id/edit_certification, and the empty state's CTA navigates to /certifications/add_certification.
  • MarketDirectory/index.tsx
    • Registers Certifications as a new FormCards entry alongside Info. It's always reported as "complete" — certifications are optional and never block the consent/publish flow.
    • Fetches the farm's certifications, system types and certifiers
  • CertificationsEmptyState
    • Added an optional className prop.

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

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)

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

@SayakaOno SayakaOno self-assigned this Jul 16, 2026
@SayakaOno SayakaOno added the enhancement New feature or request label Jul 16, 2026
@SayakaOno
SayakaOno requested review from a team as code owners July 16, 2026 17:42
@SayakaOno
SayakaOno requested review from kathyavini and removed request for a team July 16, 2026 17:42
@SayakaOno
SayakaOno added this pull request to the merge queue Jul 16, 2026
Merged via the queue into integration with commit b9d3028 Jul 16, 2026
6 checks passed
@SayakaOno

Copy link
Copy Markdown
Collaborator Author

Merged without review since we only have one dev this week.

@SayakaOno
SayakaOno deleted the LF-5380/Add_certifications_section_to_market_directory branch July 16, 2026 19:03
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