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

Missing Strings #25954

Merged
merged 1 commit into from Aug 21, 2019
Merged

Missing Strings #25954

merged 1 commit into from Aug 21, 2019

Conversation

brianteeman
Copy link
Contributor

The following keys had no translation
JLIB_LOGIN_AUTHORISATION="Your access has been authorised."
JLIB_LOGIN_DENIED="Your access has been denied."
JLIB_LOGIN_EXPIRED="Your authentication has expired."

From the relevant section in the codebase it is not very clear exactly what they should be

  	// Validate that the user should be able to login (different to being authenticated).
  	// This permits authentication plugins blocking the user

So these are fairly generic and the plugin creator can override them if they want. Nothing in core will trigger them to be displayed so I wouldn't waste much time on them.

Pull Request for Issue #25930 .

Testing Instructions

Code review only

The following keys had no translation
JLIB_LOGIN_AUTHORISATION="Your access has been authorised."
JLIB_LOGIN_DENIED="Your access has been denied."
JLIB_LOGIN_EXPIRED="Your authentication has expired."

From the relevant section in the codebase it is not very clear exactly what they should be

			// Validate that the user should be able to login (different to being authenticated).
			// This permits authentication plugins blocking the user

So these are fairly generic and the plugin creator can override them if they want. Nothing in core will trigger them to be displayed so I wouldn't waste much time on them.
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging labels Aug 21, 2019
@alikon alikon mentioned this pull request Aug 21, 2019
@infograf768
Copy link
Member

I have tested this item ✅ successfully on 1fa6d10


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25954.

1 similar comment
@alikon
Copy link
Contributor

alikon commented Aug 21, 2019

I have tested this item ✅ successfully on 1fa6d10


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25954.

@joomla-cms-bot joomla-cms-bot removed Language Change This is for Translators PR-staging labels Aug 21, 2019
@alikon
Copy link
Contributor

alikon commented Aug 21, 2019

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/25954.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 21, 2019
@infograf768 infograf768 merged commit ac43911 into joomla:staging Aug 21, 2019
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-staging and removed RTC This Pull Request is Ready To Commit labels Aug 21, 2019
@infograf768 infograf768 added this to the Joomla 4.0 milestone Aug 21, 2019
@infograf768
Copy link
Member

tks

@alikon
Copy link
Contributor

alikon commented Aug 21, 2019

wow less than 1 hour 👍

@infograf768
Copy link
Member

Let's get this one in too:
#25956

@brianteeman
Copy link
Contributor Author

Thanks

@brianteeman brianteeman deleted the strings branch August 21, 2019 10:55
@brianteeman
Copy link
Contributor Author

@infograf768 can you correct the Milestone please

@infograf768
Copy link
Member

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants