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

FIX Update behat toast logic so it works with quotes #9770

Conversation

maxime-rainville
Copy link
Contributor

@maxime-rainville maxime-rainville commented Nov 15, 2020

Toast notification will often contain double quotes. This PR update our logic for testing toast notification in behat so it allows messages with double quotes.

@Cheddam
Copy link
Member

Cheddam commented Nov 15, 2020

@maxime-rainville Might be best to hold off on merging this until 4.7.0@stable is out and then re-target it for release in 4.7.1.

@maxime-rainville maxime-rainville changed the base branch from 4 to 4.7 November 15, 2020 21:45
@maxime-rainville maxime-rainville force-pushed the pulls/4/allows-test-for-toast-with-quotes branch from b7b1c72 to c042f83 Compare November 15, 2020 21:50
@maxime-rainville
Copy link
Contributor Author

I've retargetted the PR to the 4.7 branch.

@maxime-rainville maxime-rainville force-pushed the pulls/4/allows-test-for-toast-with-quotes branch from c042f83 to 92af6b3 Compare January 19, 2021 02:16
@maxime-rainville maxime-rainville merged commit 1a75efa into silverstripe:4.7 Jan 19, 2021
@maxime-rainville maxime-rainville deleted the pulls/4/allows-test-for-toast-with-quotes branch January 19, 2021 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants