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

[12.0][FIX] shopinvader guest mode export #484

Conversation

lmignon
Copy link
Collaborator

@lmignon lmignon commented Oct 30, 2019

foward port of #465

@lmignon lmignon added the 12.0 label Oct 30, 2019
@lmignon lmignon mentioned this pull request Oct 30, 2019
77 tasks


class TestShopinvaderPartnerGuest(CommonShopinvaderPartner):
def setUp(self, *args, **kwargs):
Copy link
Contributor

Choose a reason for hiding this comment

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

not blocking here but we should stop putting global create/write on setUp -> we get tons of DB operations that are not needed for every single test ;)

@lmignon
Copy link
Collaborator Author

lmignon commented Oct 30, 2019

/ocabot merge patch

@shopinvader-git-bot
Copy link

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 12.0-ocabot-merge-pr-484-by-lmignon-bump-patch, awaiting test results.

shopinvader-git-bot pushed a commit that referenced this pull request Oct 30, 2019
Signed-off-by lmignon
@shopinvader-git-bot shopinvader-git-bot merged commit a81d621 into shopinvader:12.0 Oct 30, 2019
@shopinvader-git-bot
Copy link

Congratulations, your PR was merged at 3f9eb1d. Thanks a lot for contributing to shopinvader. ❤️

@lmignon lmignon deleted the 12.0-fix_shopinvader_guest_mode_export branch October 31, 2019 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants