Skip to content

Conversation

@rpavlini
Copy link
Collaborator

@rpavlini rpavlini commented Jan 12, 2026

Description

  • api key table & create dialog updates
  • adds mutations/queries
  • adds form & validation libraries for improved form handling

Documentation

  • This change requires a documentation update
  • I have made corresponding changes to the documentation

Related Issue(s)

closes #2973
closes DAS-12
closes DAS-14

Screenshots

Before After
Screenshot 2026-01-12 at 20 46 17 Screenshot 2026-01-12 at 16 05 18
Screenshot 2026-01-12 at 20 46 12 Screenshot 2026-01-12 at 16 05 03

@linear
Copy link

linear bot commented Jan 12, 2026

@rpavlini rpavlini force-pushed the feat/api-keys-updates branch from e6b1fb0 to 12c38b6 Compare January 13, 2026 10:44
@rpavlini rpavlini requested a review from Tpuljak January 13, 2026 13:51
@rpavlini rpavlini marked this pull request as ready for review January 13, 2026 13:51
@rpavlini rpavlini force-pushed the feat/api-keys-updates branch from bf55ae4 to 1bbf9c8 Compare January 19, 2026 18:01
Copy link
Member

Choose a reason for hiding this comment

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

Screenshot 2026-01-20 at 09 25 43

I don't like that the toggled (active) state is exactly the same style as the hover state.

On the screenshot above, I have 2 perms active and one is hovered. Tell me which is which 😄

<Alert variant="info">
<InfoIcon />
<AlertTitle>Sandboxes Access</AlertTitle>
<AlertDescription>This key grants access to the Sandboxes resource.</AlertDescription>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<AlertDescription>This key grants access to the Sandboxes resource.</AlertDescription>
<AlertDescription>This key grants read and write to the Sandboxes resource.</AlertDescription>

I suggest being explicit here because "grants access" might be interpreted as read-only

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated!

Signed-off-by: rpavlini <rpavlinic@daytona.io>
Signed-off-by: rpavlini <rpavlinic@daytona.io>
Signed-off-by: rpavlini <rpavlinic@daytona.io>
Signed-off-by: rpavlini <rpavlinic@daytona.io>
Signed-off-by: rpavlini <rpavlinic@daytona.io>
Signed-off-by: rpavlini <rpavlinic@daytona.io>
Signed-off-by: rpavlini <rpavlinic@daytona.io>
Signed-off-by: rpavlini <rpavlinic@daytona.io>
Signed-off-by: rpavlini <rpavlinic@daytona.io>
@rpavlini rpavlini force-pushed the feat/api-keys-updates branch from 1bbf9c8 to 97b91c6 Compare January 20, 2026 16:01
@rpavlini rpavlini requested a review from Tpuljak January 20, 2026 16:48
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.

Improve UX for checking API key permissions

3 participants