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
[API] Resend verification email refactor to take current user from the security context #12978
Conversation
lchrusciel
commented
Aug 18, 2021
Q | A |
---|---|
Branch? | master |
Bug fix? | no |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Related tickets | |
License | MIT |
16b2ed3
to
cf84fc6
Compare
|
||
/** @experimental */ | ||
class ResendVerificationEmail implements ChannelCodeAwareInterface, LocaleCodeAwareInterface, IriToIdentifierConversionAwareInterface | ||
class ResendVerificationEmail implements ShopUserIdAwareInterface, ChannelCodeAwareInterface, LocaleCodeAwareInterface, IriToIdentifierConversionAwareInterface |
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.
class ResendVerificationEmail implements ShopUserIdAwareInterface, ChannelCodeAwareInterface, LocaleCodeAwareInterface, IriToIdentifierConversionAwareInterface | |
final class ResendVerificationEmail implements ShopUserIdAwareInterface, ChannelCodeAwareInterface, LocaleCodeAwareInterface, IriToIdentifierConversionAwareInterface |
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.
I agree, that this approach would be more proper from the clean code perspective, but we've decided to leave commands open, as they are simple DTOs and it makes it easier to extend basic logic
src/Sylius/Bundle/ApiBundle/Command/Account/ResendVerificationEmail.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Command/Account/ResendVerificationEmail.php
Outdated
Show resolved
Hide resolved
cf84fc6
to
d9388d6
Compare
src/Sylius/Bundle/ApiBundle/Command/Account/ResendVerificationEmail.php
Outdated
Show resolved
Hide resolved
1a25a4a
to
736a6c8
Compare
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.
But the rebase is needed
src/Sylius/Bundle/ApiBundle/Resources/config/api_resources/VerifyCustomerAccount.xml
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/Resources/config/serialization/Taxon.xml
Outdated
Show resolved
Hide resolved
473b46e
to
770e1a2
Compare
…e security context
770e1a2
to
f247e3e
Compare
Thanks, Łukasz! 🎉 |