Skip to content

Fix/970 move farm contact to left menu - #1012

Open
yjm7vj wants to merge 2 commits into
mainfrom
fix/970-move-farm-contact-to-left-menu
Open

Fix/970 move farm contact to left menu#1012
yjm7vj wants to merge 2 commits into
mainfrom
fix/970-move-farm-contact-to-left-menu

Conversation

@yjm7vj

@yjm7vj yjm7vj commented Jul 10, 2026

Copy link
Copy Markdown

Description

Fixes #970

Moved farm name and primary contact details into the account side menu, removed the old account header from the account shell, and fixed /account routing so the account landing page is protected like the nested account pages.

Changes by file:
apps/site/src/app/(authenticated)/(shell)/(accounts)/account/(with-shell)/layout.tsx
Removed the account header from the account layout.
Passed farm/contact shell data into AccountSideMenu.
Updated the loading skeleton to match the headerless layout.

apps/site/src/app/(authenticated)/(shell)/(accounts)/account/components/account-side-menu/account-side-menu.tsx
Added farm name and contact details to the top of the account side menu.
Updated the component to accept typed props.

apps/site/src/app/(authenticated)/(shell)/(accounts)/account/components/account-side-menu/types.ts
Added AccountSideMenuProps.

apps/site/src/app/(authenticated)/(shell)/(accounts)/account/components/account-side-menu/account-side-menu.test.tsx
Updated tests to pass the new side menu props.
Added coverage for rendered farm/contact details.

apps/site/src/app/(authenticated)/(shell)/(accounts)/account/db.ts
Extended getAccountShellData() to return farm name plus contact name, email, and phone.

apps/site/src/proxy/auth.ts
Added /account as an exact protected route so the account landing page works correctly.

apps/site/src/proxy/auth.test.ts
Updated route protection expectations for /account.

packages/db/scripts/seed-local-db.ts
Made OpenAI embedding import lazy so local DB seeding can run without OPENAI_EMBEDDINGS_KEY.

Checklist

  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • Any components that you've modified are accessible.
  • You've used conventional commits where appropriate

@vercel

vercel Bot commented Jul 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
internal-dashboard Ready Ready Preview, Comment Jul 10, 2026 4:32pm
nightcrawler Ready Ready Preview, Comment Jul 10, 2026 4:32pm

@vincenttodd

vincenttodd commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

the add info account side menu was done in a different pr and this issue was not updated. Removing the old account header (the Todd workmark is, but i don't think the home or farm name are part of the header) and updating account routing still stands. Either you stirred something up or something was added to the account menu section. The second image is what it should look like still (don't worry about the left menu stuff)

Screenshot 2026-07-10 at 4 30 55 PM Screenshot_2026-07-10_at_4 39 38_PM

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.

Move farm name and contact to left menu and remove header

2 participants