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

Some overlay improvements #15032

Merged
merged 1 commit into from Nov 4, 2019
Merged

Some overlay improvements #15032

merged 1 commit into from Nov 4, 2019

Conversation

tsteur
Copy link
Member

@tsteur tsteur commented Oct 21, 2019

refs #14835

First fix improves the loading when you have eg configured

https://fooBar.com as a site url, and the domain in the url is foobar.com

The second fix is event.data related where in this case a website was firing an event and the site was posting event.data as an object

They don't fix the actual issue but make it more reliable

refs #14835 

First fix improves the loading when you have eg configured

https://fooBar.com as a site url, and the domain in the url is foobar.com

The second fix is event.data related where in this case a website was firing an event and the site was posting event.data as an object
@tsteur tsteur added Pull Request WIP Indicates the current pull request is still work in progress and not ready yet for a review. Needs Review PRs that need a code review and removed Pull Request WIP Indicates the current pull request is still work in progress and not ready yet for a review. labels Oct 21, 2019
@tsteur tsteur modified the milestones: 3.12.0, 3.13.0 Oct 21, 2019
Copy link
Member

@sgiehl sgiehl left a comment

Choose a reason for hiding this comment

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

Changes look legit and tests are still working so 👍

@diosmosis diosmosis merged commit b11c097 into 3.x-dev Nov 4, 2019
@diosmosis diosmosis deleted the 14835_1 branch November 4, 2019 00:14
diosmosis pushed a commit that referenced this pull request Nov 6, 2019
refs #14835 

First fix improves the loading when you have eg configured

https://fooBar.com as a site url, and the domain in the url is foobar.com

The second fix is event.data related where in this case a website was firing an event and the site was posting event.data as an object
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants