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

When token_auth is re-generated, remove wording that user will be logged out #13606

Merged
merged 1 commit into from Nov 24, 2018

Conversation

tsteur
Copy link
Member

@tsteur tsteur commented Oct 14, 2018

fix #13597

@tsteur tsteur added not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. Needs Review PRs that need a code review labels Oct 14, 2018
@tsteur tsteur added this to the 3.7.0 milestone Oct 14, 2018
"TokenRegenerateConfirm": "Changing the API authentication token will invalidate the user's current token. If the user is currently logged in, they may be required to log in again. Do you really want to continue?",
"TokenRegenerateConfirmSelf": "Changing the API authentication token will invalidate your own token. You may be required to log in again if you proceed. Do you really want to change your authentication token?",
"TokenRegenerateLogoutWarning": "When you re-generate your token you will be logged out and need to log in again.",
"TokenRegenerateConfirmSelf": "Changing the API authentication token will invalidate your own token. If the current token is in use, you need to update all API clients with the newly generated token. Do you really want to change your authentication token?",
Copy link
Member

Choose a reason for hiding this comment

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

Not sure if this changed, but do we need to change the key here since the translation content is different?

Copy link
Member Author

Choose a reason for hiding this comment

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

AFAIK it is not needed anymore @sgiehl

Copy link
Member

Choose a reason for hiding this comment

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

Not needed. Tranisfex will automatically clear all translations of this key if the content changes,

@sgiehl sgiehl merged commit 4c92e54 into 3.x-dev Nov 24, 2018
@sgiehl sgiehl deleted the 13597 branch November 24, 2018 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When token_auth is re-generated, remove wording that user will be logged out
3 participants