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

Skip to content

[Product Review] fixed review validation when edited by admin#10118

Merged
pamil merged 1 commit into
Sylius:1.2from
kboduch:issue_9895
Jan 25, 2019
Merged

[Product Review] fixed review validation when edited by admin#10118
pamil merged 1 commit into
Sylius:1.2from
kboduch:issue_9895

Conversation

@kboduch
Copy link
Copy Markdown
Contributor

@kboduch kboduch commented Jan 25, 2019

Q A
Branch? 1.3
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets #9895
License MIT

The test was a false-positive. I've changed it to create Customer and ShopUser so that UniqueReviewerEmailValidator has a user to validate against.

@kboduch kboduch requested a review from a team as a code owner January 25, 2019 00:29
@pamil pamil changed the base branch from 1.3 to 1.2 January 25, 2019 11:37
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Jan 25, 2019

The base of this pull-request was changed, you need fetch and reset your local branch
if you want to add new commits to this pull request. Reset before you pull, else commits
may become messed-up.

Unless you added new commits (to this branch) locally that you did not push yet,
execute git fetch origin && git reset "issue_9895" to update your local branch.

Feel free to ask for assistance when you get stuck 👍

@pamil pamil added Potential Bug Potential bugs or bugfixes, that needs to be reproduced. Admin AdminBundle related issues and PRs. labels Jan 25, 2019
@pamil pamil merged commit 042c9c0 into Sylius:1.2 Jan 25, 2019
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Jan 25, 2019

Thanks, Krzysztof! 🥇

@kboduch
Copy link
Copy Markdown
Contributor Author

kboduch commented Jan 25, 2019

It was a pleasure, I've learned a lot about behat from the sylius project. I'm glad I could help out.

@kboduch kboduch deleted the issue_9895 branch January 25, 2019 11:45
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. Potential Bug Potential bugs or bugfixes, that needs to be reproduced.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants