-
Notifications
You must be signed in to change notification settings - Fork 11k
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
[FIX] Missing string 'There_are_no_applications' on the OAuth Apps Page #9626
Conversation
@@ -1864,6 +1864,7 @@ | |||
"theme-custom-css": "Custom CSS", | |||
"theme-font-body-font-family": "Body Font Family", | |||
"There_are_no_agents_added_to_this_department_yet": "There are no agents added to this department yet.", | |||
"There_are_no_applications": "There are no OAuth Applications added yet.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder, would it make more sense to have it worded: "No oAuth Applications have been added yet."? I'm not 100% sure 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Definitely makes more sense. I’m gonna change that later. Should I change the variable name as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so, should be fine with how it currently is. Thanks!
Sorry for this mess of a PR, but I think I'm done :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fix and making the update! :)
@RocketChat/core
Closes #9627
Sorry for any mistakes. This is my first PR ever :)