-
Notifications
You must be signed in to change notification settings - Fork 67
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
Parameterize app name in web interface #3756
Parameterize app name in web interface #3756
Conversation
…your editor first.
…eCI. Trying to fix that.
I want to do a round of spot-checking on this locally in the browser before I ask for a code review. |
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.
Assuming nothing breaks, this looks ok to me. We can play around in the staging envs as well.
It's not worth changing here, but it's worth evaluating whether or not we need the app name at all in a lot of the client-facing messaging, we probably could have just dropped it in a bunch of places. 🤷
…alist-web Parameterize app name in web interface
Changes proposed in this pull request:
uaaInvite.js
which is only relevant to Pages)This is work toward #3709
security considerations
None