Skip to content

feat: change org member roles, tenant tag improvements - #4440

Merged
abelanger5 merged 2 commits into
mainfrom
belanger/org-member-role-changes
Jul 15, 2026
Merged

feat: change org member roles, tenant tag improvements#4440
abelanger5 merged 2 commits into
mainfrom
belanger/org-member-role-changes

Conversation

@abelanger5

Copy link
Copy Markdown
Contributor

Description

Two improvements to the settings pages:

  1. Allows organization member roles to be modified by owners
  2. Places tenant tags in the tenant's settings page so it's easier to discover how to edit them.

Type of change

  • New feature (non-breaking change which adds functionality)

What's Changed

Org member modal:

image

Tags:

image

Checklist

Changes have been:

  • Tested (unit, integration, or manually with steps specified) - manually
  • Linted and formatted

🤖 AI Disclosure
  • I acknowledge that an LLM was used in the creation of this Pull Request, in accordance with Hatchet's AI_POLICY.md.
  • Details: Claude Fable

@vercel

vercel Bot commented Jul 15, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
hatchet-docs Ready Ready Preview, Comment Jul 15, 2026 9:47pm

Request Review

@github-actions github-actions Bot added the dashboard Related to the Hatchet dashboard label Jul 15, 2026
@abelanger5
abelanger5 requested a review from Copilot July 15, 2026 20:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances the settings UX by (1) adding a control-plane-backed UI flow for organization owners to change member roles, and (2) surfacing tenant tags directly on the tenant’s General settings tab (with an edit entry point) to make tag discovery/editing easier.

Changes:

  • Add an EditMemberRoleModal and wire it into the org members table actions (control-plane only).
  • Add a TenantTags setting row on the tenant General page, including an “Edit tags” modal entry point.
  • Extend the organization API wrapper and generated control-plane client to support updating an org member’s role.

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
frontend/app/src/pages/organizations/$organization/components/edit-member-role-modal.tsx New modal UI + mutation wiring for updating an org member’s role.
frontend/app/src/pages/main/v1/tenant-settings/overview/index.tsx Adds “Tags” row on tenant General settings with tag display + edit modal launch.
frontend/app/src/pages/main/v1/tenant-settings/organization/index.tsx Wires “Change role” action into member dropdown and renders the new role-edit modal.
frontend/app/src/lib/api/organization-wrapper.ts Adds a control-plane-only mutation wrapper for organization member role updates.
frontend/app/src/lib/api/generated/control-plane/data-contracts.ts Adds the generated request contract for updating an org member role.
frontend/app/src/lib/api/generated/control-plane/Api.ts Adds the generated PATCH endpoint client for updating an org member’s role.

Comment thread frontend/app/src/pages/main/v1/tenant-settings/overview/index.tsx
@github-actions github-actions Bot added the engine Related to the core Hatchet engine label Jul 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Optional test failure: The load-deadlock job failed on this PR (optimistic-scheduling=false). This check is non-mandatory and does not block merging, but may be worth investigating. View logs

@abelanger5
abelanger5 merged commit c46ed82 into main Jul 15, 2026
58 checks passed
@abelanger5
abelanger5 deleted the belanger/org-member-role-changes branch July 15, 2026 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dashboard Related to the Hatchet dashboard engine Related to the core Hatchet engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants