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

Auto-post of schema #53

Merged
merged 2 commits into from Oct 9, 2018
Merged

Auto-post of schema #53

merged 2 commits into from Oct 9, 2018

Conversation

stevector
Copy link
Contributor

https://www.drupal.org/node/2775549

Because this bug manifests on the creation of new multidevs, it is hard to test without significantly slowly down the test suite. Not sure what to do about that.

@stevector
Copy link
Contributor Author

Testing of this feature could be done by copying from one multidev to another: https://github.com/pantheon-systems/search_api_pantheon/pull/36/files

But again, I'm not sure if it is worth the additional time per build.

@arosboro
Copy link

@stevector This looks like it would work in theory. Are you using it on Pantheon with any projects successfully?

@stevector
Copy link
Contributor Author

@arosboro, I don't have any sites that need it / use it. Let me know if you can test the PR with a real site.

@dalin-
Copy link

dalin- commented Jul 9, 2018

I've set this up on three separate D8 sites. I'll report back if there's any adverse affects.

@NickDickinsonWilde
Copy link

I can confirm that this fixes multidev sites to work without messing around. 👍

@stevector stevector changed the title (WIP) auto-post of schema Auto-post of schema Oct 9, 2018
@stevector stevector merged commit cf291ef into 8.x-1.x Oct 9, 2018
@stevector stevector deleted the 2775549-auto-post-schema branch October 9, 2018 02:17
@stevector
Copy link
Contributor Author

Thanks @arosboro, @dalin, @NickWilde1990!

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.

None yet

4 participants