Closed NoRi2909 closed 2 days ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@googanhiem I like the change. What do you think?
Open suggestion: It may suit better nowadays under the Settings page (/config/dashboard) where we can give it a proper name and description?
EDIT: Right, bad suggestion, this is only shown for users that don't have admin rights and hence no "Settings".
For non-admin user accounts the 'Settings' item in the sidebar is hidden in the browser and replaced with 'App configuration'.
That creates an inconsistency and introduces a very technical term here where we're targeting the non-techy users that don't have admin privileges.
Proposed change
Replace "App configuration" with "App settings". That is shown instead of "Settings" for non-admins.
On mobile this is also the usual term in other apps.
Type of change
Additional information
Checklist
If user exposed functionality or configuration variables are added/changed: