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

Validate resending order confirmation email #15735

Merged
merged 5 commits into from Feb 2, 2024

Conversation

TheMilek
Copy link
Member

Q A
Branch? 1.13
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
License MIT

Continuation of #15712 with validation added

@TheMilek TheMilek added the Admin AdminBundle related issues and PRs. label Jan 16, 2024
@TheMilek TheMilek requested a review from a team as a code owner January 16, 2024 12:06
@probot-autolabeler probot-autolabeler bot added API APIs related issues and PRs. Maintenance CI configurations, READMEs, releases, etc. labels Jan 16, 2024
Copy link

github-actions bot commented Jan 16, 2024

Bunnyshell Preview Environment deleted

Available commands:

  • /bns:deploy to redeploy the environment

Wojdylak
Wojdylak previously approved these changes Jan 17, 2024
@NoResponseMate NoResponseMate dismissed Wojdylak’s stale review January 17, 2024 13:02

The merge-base changed after approval.

@NoResponseMate NoResponseMate force-pushed the validate-order-states branch 3 times, most recently from e445565 to ace32d0 Compare February 1, 2024 12:54
Wojdylak
Wojdylak previously approved these changes Feb 1, 2024
Rafikooo
Rafikooo previously approved these changes Feb 1, 2024
@jakubtobiasz jakubtobiasz merged commit 0627f0a into Sylius:1.13 Feb 2, 2024
28 checks passed
@jakubtobiasz
Copy link
Member

Thank you, @TheMilek!

@TheMilek TheMilek deleted the validate-order-states branch May 7, 2024 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin AdminBundle related issues and PRs. API APIs related issues and PRs. Maintenance CI configurations, READMEs, releases, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants