Skip to content

feat(settings): Company Profile settings page, logo upload, validation (issue #418)#462

Merged
Damidesign merged 1 commit into
Navin-xmr:mainfrom
MAN7A-afk:main
Jun 28, 2026
Merged

feat(settings): Company Profile settings page, logo upload, validation (issue #418)#462
Damidesign merged 1 commit into
Navin-xmr:mainfrom
MAN7A-afk:main

Conversation

@MAN7A-afk

Copy link
Copy Markdown
Contributor

Overview
Closes #418

Adds a Company Profile settings page so companies can manage their public profile and branding (logo) inside the app instead of contacting support.
What I changed

Added: Company profile UI + test
frontend/src/pages/Settings/CompanyProfile/CompanyProfile.tsx
frontend/src/pages/Settings/CompanyProfile/CompanyProfile.test.tsx
Updated:
Integrated tab: frontend/src/pages/Settings/Settings.tsx
Navbar now shows saved company logo:

Files touched

Added: frontend/src/pages/Settings/CompanyProfile/CompanyProfile.tsx
Added: frontend/src/pages/Settings/CompanyProfile/CompanyProfile.test.tsx
Modified: frontend/src/pages/Settings/Settings.tsx
Modified: frontend/src/components/Navbar/Navbar.tsx

Acceptance criteria checklist

All fields editable and saved per section: implemented and persisted to localStorage.
Logo upload, crop, and preview: implemented client-side, preview before save.
Validation errors show inline: implemented per-field with messages.
Unsaved changes indicator per section: implemented (dot indicator in section header).
Saved logo appears in Navbar: implemented (reads navin-company-logo on route change).

How to test locally

Start dev server:
Open Settings → select the new "Company Profile" tab (company-only tab).
Try editing sections, observe unsaved dot, click per-section Save.
Upload an image via the Branding section and Save — logo should appear in the Navbar.

@Damidesign
Damidesign merged commit b5126cc into Navin-xmr:main Jun 28, 2026
2 of 5 checks passed
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.

1 participant