Skip to content

Feature/org design fixes#1787

Draft
Shreyag02 wants to merge 5 commits into
mainfrom
feature/org-design-fixes
Draft

Feature/org design fixes#1787
Shreyag02 wants to merge 5 commits into
mainfrom
feature/org-design-fixes

Conversation

@Shreyag02

Copy link
Copy Markdown
Contributor

Summary

Changes

Technical Details

Test Plan

  • Manual testing completed
  • Build and type checking passes

SQL Safety (if your PR touches *_repository.go or goqu.*)

  • Values flow through ? placeholders, goqu.Ex{}, or goqu.Record{} — never fmt.Sprintf or + building a query that gets executed.
  • ToSQL() callers capture and forward params (query, params, err := stmt.ToSQL(); db.…Context(ctx, …, query, params...)). Never query, _, err := ….
  • No ? placeholders inside single-quoted SQL literals in goqu.L (use make_interval(hours => ?)-style functions instead).
  • Any //nolint:forbidigo or // #nosec G20x annotation has a one-line justification on the same line that a reviewer can verify.

Hide the "New Organization" button (and its separator) while search is
active so the listing navbar stays uncluttered, and align the navbar
height with the org-details navbar (48px) instead of 56px.
Display org member roles as "Owner", "Member", "Admin" instead of "Organization Owner" etc. in the Role column, via a formatRoleTitle helper.
Disable the "Add member" project action when the org has only one member,
since there is no one else to search for and add. Also keep the project
row highlighted while its action menu is open.
Distinguish an empty tab (no data at all) from a no-match search result:
add a dedicated zero state to the members, projects, invoices, tokens and
API tabs via the DataTable zeroState prop, so the "no matches for that
keyword" copy only appears when a search or filter is active. Table headers
are hidden in both states by the DataTable.
@vercel

vercel Bot commented Jul 21, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
frontier Ready Ready Preview, Comment Jul 21, 2026 11:25am

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