Skip to content

[4.0] Admin Warning when no cookies#26192

Merged
wilsonge merged 8 commits intojoomla:4.0-devfrom
brianteeman:leeds3
Sep 8, 2019
Merged

[4.0] Admin Warning when no cookies#26192
wilsonge merged 8 commits intojoomla:4.0-devfrom
brianteeman:leeds3

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

@brianteeman brianteeman commented Sep 6, 2019

PR for #26037

Summary of Changes

You can not login to the Joomla admin if you have cookies disabled in your browser but we never tell the user that. The login just doesnt work without any error message
This tests to see if cookies are enabled in the browser and displays an alert and message if not

Testing Instructions

Disable cookies in your web browser and try to login to the joomla admin

Expected result

Either successful login or an error message

Actual result

nothing

You can not login to the Joomla admin if you have cookies disabled in your browser but we never tell the user that. The login just doesnt work without any error message
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-4.0-dev labels Sep 6, 2019
@joomla-cms-bot joomla-cms-bot added the NPM Resource Changed This Pull Request can't be tested by Patchtester label Sep 7, 2019
Co-Authored-By: zero-24 <zero-24@users.noreply.github.com>
@wilsonge wilsonge merged commit eaa4f48 into joomla:4.0-dev Sep 8, 2019
@wilsonge
Copy link
Copy Markdown
Contributor

wilsonge commented Sep 8, 2019

Thanks!

@wilsonge wilsonge added this to the Joomla 4.0 milestone Sep 8, 2019
@brianteeman
Copy link
Copy Markdown
Contributor Author

thanks all

@brianteeman brianteeman deleted the leeds3 branch September 8, 2019 12:15
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 NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants