Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Finalize Account Settings and switch to a dedicated page #681

Merged
merged 6 commits into from
Jan 26, 2025

Conversation

tjementum
Copy link
Member

Summary & Motivation

Convert the Account Settings from a modal dialog accessed through the AvatarButton to a standalone page, now accessible via the Account menu in the Side Menu. This change improves usability and provides a more structured experience for managing account details.

A new endpoint has been implemented to update the tenant name (account name) when saving changes.

Previously, all tenant-related API endpoints used /tenants/{id}, but these have been updated to /tenants/current for improved clarity and consistency. Both GET and PUT endpoints now use this new convention. To support this change, a new GetCurrentTenantAsync method has been added to the TenantRepository, and the GetTenant and GetCurrentTenant commands have been updated to use this method.

All related frontend components, commands, and tests have been updated accordingly.

Checklist

  • I have added tests, or done manual regression tests
  • I have updated the documentation, if necessary

@tjementum tjementum added the Enhancement New feature or request label Jan 26, 2025
@tjementum tjementum self-assigned this Jan 26, 2025
Copy link

linear bot commented Jan 26, 2025

@tjementum tjementum merged commit 6949fb3 into main Jan 26, 2025
9 checks passed
@tjementum tjementum deleted the pp-313-finalize-account-settings branch January 26, 2025 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant