-
Notifications
You must be signed in to change notification settings - Fork 32
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
Fix behavior when cloning region with page based offers #2667
Conversation
One test i failing becuase the valid link count is somehow higher now after duplicating a region - I don't really know why, does someone have an idea? |
With this pr the following link is classified as valid:
I assume that is now the case with this pr, while it had no offers before, which causes it to mark the link as valid... So the simple fix to just update the expected numbers should be fine I think. |
b8bd76e
to
48cfcd6
Compare
Code Climate has analyzed commit 12c48c6 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 88.8% (50% is the threshold). This pull request will bring the total coverage in the repository to 81.7% (0.0% change). View more on Code Climate. |
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.
LGTM
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.
Thank you 👍 works as expected!
48cfcd6
to
12c48c6
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.
🎉
Short description
Currently, cloning a region which has embedded offfers into pages can lead to unexpected errors when the newly-created region does not have the same offers enabled.
Proposed changes
Side effects
Resolved issues
Fixes: #2665
Pull Request Review Guidelines