-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Login feedback if user do not have offline access #39561
Login feedback if user do not have offline access #39561
Conversation
@carlitorweb As the check added by this PR is in the |
@richard67 For API dont matter. Right now, an unique Bearer token or credentials need be provided in the request. The later have his own separate routine for authenticate the user I did not interact much with the CLI yet, but I tested for example, when offline the site I could add a new user trought the CLI without problem. I not think so far this change affect the CLI application. Anyway, let me kown if anything need be changed. |
@richard67 can you run the drone again, not sure why fail |
I have tested this item ✅ successfully on 0f48b71 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39561. |
1 similar comment
I have tested this item ✅ successfully on 0f48b71 This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39561. |
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39561. |
Thank you for making it easier for the users! |
Pull Request for Issue #39559
I used the core string
JLIB_LOGIN_DENIED = Your access has been denied
. But I think we should create a new one just for this messageSummary of Changes
PlgAuthenticationJoomla
verified is a valid user who try to access, but not check the permission of the offline mode.Testing Instructions
Set your site in offline mode. Go to the frontend and try login with a user who have the permission Offline Access = Not allowed
Actual result BEFORE applying this Pull Request
The user cant login, but not message show up of why cant login
Expected result AFTER applying this Pull Request
The user cant login, but a message show up of why cant login