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

Refacto login forms and add Employee CQRS commands #36162

Merged
merged 10 commits into from
May 24, 2024

Conversation

jolelievre
Copy link
Contributor

@jolelievre jolelievre commented May 15, 2024

Questions Answers
Branch? develop
Description? Refacto login forms
Type? refacto
Category? BO
BC breaks? no
Deprecations? no
How to test? UI tests and ci green
UI Tests https://github.com/jolelievre/ga.tests.ui.pr/actions/runs/9193722916
Fixed issue or discussion? Fixes #36159
Related PRs ~
Sponsor company ~

@prestonBot prestonBot added develop Branch Refactoring Type: Refactoring labels May 15, 2024
@jolelievre jolelievre changed the title Refacto login forms Refacto login forms and add Employee CQRS commands May 15, 2024
@jolelievre jolelievre force-pushed the refacto-login-forms branch 5 times, most recently from f9386e9 to e5ae01d Compare May 16, 2024 17:57
@jolelievre jolelievre marked this pull request as ready for review May 16, 2024 19:01
@jolelievre jolelievre requested a review from a team as a code owner May 16, 2024 19:01
@jolelievre jolelievre force-pushed the refacto-login-forms branch 2 times, most recently from 63cdf77 to f6bb8a7 Compare May 20, 2024 07:56
tleon
tleon previously approved these changes May 21, 2024
boherm
boherm previously approved these changes May 21, 2024
Copy link
Member

@boherm boherm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but just one question about maildev :)

.github/workflows/behaviour.yml Show resolved Hide resolved
tests/Resources/MailDevClient.php Outdated Show resolved Hide resolved
@ps-jarvis ps-jarvis added the Waiting for QA Status: action required, waiting for test feedback label May 21, 2024
@jolelievre jolelievre dismissed stale reviews from boherm and tleon via 6002d87 May 22, 2024 13:13
@jolelievre jolelievre removed the Waiting for QA Status: action required, waiting for test feedback label May 22, 2024
…dType, improve validation feedback on reset form
@ps-jarvis ps-jarvis added the Waiting for QA Status: action required, waiting for test feedback label May 24, 2024
@jolelievre jolelievre removed the Waiting for QA Status: action required, waiting for test feedback label May 24, 2024
@jolelievre jolelievre merged commit 7ef6784 into PrestaShop:develop May 24, 2024
38 checks passed
@jolelievre jolelievre deleted the refacto-login-forms branch May 24, 2024 10:32
@jolelievre jolelievre added this to the 9.0.0 milestone Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
develop Branch Refactoring Type: Refactoring
Projects
Status: To be tested
5 participants