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

Allows to throw own exception messages in admin login #882

Merged

Conversation

moritzdemmer
Copy link
Contributor

Why

There is no possibility to throw your own error messages in the admin login area as the message is hard coded.
When adding new functionality to the admin login, it may be necessary to show own messages instead of the "Wrong Username/Password",

Sieg added a commit that referenced this pull request Nov 8, 2021
Related #882

Signed-off-by: Anton Fedurtsya <anton@fedurtsya.com>
Sieg added a commit that referenced this pull request Nov 8, 2021
Related #882

Signed-off-by: Anton Fedurtsya <anton@fedurtsya.com>
Sieg added a commit that referenced this pull request Nov 8, 2021
As exception message is not hardcoded and not overwritten by the controller anymore, its more precise.

Related #882

Signed-off-by: Anton Fedurtsya <anton@fedurtsya.com>
Sieg added a commit that referenced this pull request Nov 9, 2021
Related #882

Signed-off-by: Anton Fedurtsya <anton@fedurtsya.com>
Sieg added a commit that referenced this pull request Nov 9, 2021
As exception message is not hardcoded and not overwritten by the controller anymore, its more precise.

Related #882

Signed-off-by: Anton Fedurtsya <anton@fedurtsya.com>
Sieg added a commit that referenced this pull request Nov 10, 2021
Related #882

Signed-off-by: Anton Fedurtsya <anton@fedurtsya.com>
Sieg added a commit that referenced this pull request Nov 10, 2021
Related #882

Signed-off-by: Anton Fedurtsya <anton@fedurtsya.com>
@Sieg Sieg merged commit 9574994 into OXID-eSales:b-6.4.x Nov 10, 2021
@Sieg
Copy link
Member

Sieg commented Nov 10, 2021

Hello @moritzdemmer. Thanks for the improvement. Merged to b-6.4.x and up to master with several changes: 2ce41dc

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

Successfully merging this pull request may close these issues.

2 participants