Skip to content

fix: Revoke token and General Settings should be editable only by admins#830

Open
quangtuanitmo18 wants to merge 4 commits intomasterfrom
fix/admin-revoke-token-and-general-settings
Open

fix: Revoke token and General Settings should be editable only by admins#830
quangtuanitmo18 wants to merge 4 commits intomasterfrom
fix/admin-revoke-token-and-general-settings

Conversation

@quangtuanitmo18
Copy link
Copy Markdown
Contributor

Description: Revoke token and General Settings should be editable only by admins

@quangtuanitmo18 quangtuanitmo18 force-pushed the fix/admin-revoke-token-and-general-settings branch from 09d297d to 8f887df Compare January 29, 2026 12:10
{{ $t('projects.settings.general.title') }}
</div>
<form
v-if="userCanEdit"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For regular team members the page will be empty, it's not the best UI. It would be better to make inputs "disabled"

But first of all, the ability to change general settings on the API side. Could you check it? maybe it is already disabled.

/>

<div
v-if="userCanEdit"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

same thing: API is first, UI is the next phase check

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.

2 participants