Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add Sylius template event#11534

Merged
pamil merged 2 commits into
Sylius:1.7from
tavy315:patch-1
Jun 15, 2020
Merged

Add Sylius template event#11534
pamil merged 2 commits into
Sylius:1.7from
tavy315:patch-1

Conversation

@tavy315
Copy link
Copy Markdown
Contributor

@tavy315 tavy315 commented May 29, 2020

Q A
Branch? 1.7
Bug fix? no/yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets #11151
License MIT

I've seen the changes done in (186c13c), but I guess that this file was missed.

@tavy315 tavy315 requested a review from a team as a code owner May 29, 2020 18:51
@probot-autolabeler probot-autolabeler Bot added the Admin AdminBundle related issues and PRs. label May 29, 2020
@lchrusciel lchrusciel closed this Jun 1, 2020
@lchrusciel lchrusciel reopened this Jun 1, 2020
@tavy315
Copy link
Copy Markdown
Contributor Author

tavy315 commented Jun 5, 2020

Can this be merged?

@lchrusciel
Copy link
Copy Markdown
Contributor

Hey @tavy315,

thanks a lot for your contribution and your patience. It can be merged to the master, however we have a red build there. Once it is fixed I will ping you back

@tavy315
Copy link
Copy Markdown
Contributor Author

tavy315 commented Jun 8, 2020

Ok, no problem. Not sure if I can help there.
By the way, targeted branch was 1.7 since all the changes were also merged into that branch, see 186c13c

@lchrusciel
Copy link
Copy Markdown
Contributor

My mistake should be merged to 1.7

@lchrusciel
Copy link
Copy Markdown
Contributor

Hey Octav, could you rebase your work to branch 1.7?

@tavy315
Copy link
Copy Markdown
Contributor Author

tavy315 commented Jun 10, 2020

@lchrusciel, sorry, I think I merged it instead. I'm still not used to sync and rebase a fork.

@pamil pamil merged commit a6ef5bf into Sylius:1.7 Jun 15, 2020
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Jun 15, 2020

Thank you, Octav! 🥇

@tavy315 tavy315 deleted the patch-1 branch June 25, 2020 18:34
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants