-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Issue 25268 #25728
Issue 25268 #25728
Conversation
Fix Test to lock database with long name
…and change exception handle logic
`UserLockedException` extends `AuthenticationException` and provides the same message. These lines were useless.
Fix Test to lock database with long name
…and change exception handle logic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- First of all - there is a lot not related changes. Could you force push only needed changes?
- I don't see any difference in order model, looks like you missed to push them
@rajeebkumar-sahoo , I am closing this PR now due to inactivity. |
Hi @rajeebkumar-sahoo, thank you for your contribution! |
resolved issue#25268: order get customer returns null from observer
Description (*)
This merge request resolves issue#25268 (order get customer returns null from observer)
Fixed Issues (if relevant)
Manual testing scenarios (*)
Pre-requisite:
i. setup:di:compile after getting the new code
ii. clean the configuration cache after adding event observer
Questions or comments
Contribution checklist (*)