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

Experiment with reusing the gutenberg store in the block manager #215

Closed
wants to merge 1 commit into from

Conversation

Tug
Copy link
Contributor

@Tug Tug commented Nov 1, 2018

In this PR we're removing the whole store folder and making the appropriate changes to our components for it to work with the gutenberg store. It’s probably not good to have an hybrid solution (where we use both stores).

WIP

Gutenberg changes: WordPress/gutenberg#11486

@Tug Tug added [Type] Enhancement Improves a current area of the editor [Status] Not Ready For Review labels Nov 1, 2018
@Tug Tug self-assigned this Nov 1, 2018
@Tug Tug force-pushed the try/experiment-state branch 2 times, most recently from aa80db3 to 33338ec Compare November 5, 2018 10:45
@Tug Tug force-pushed the try/experiment-state branch 7 times, most recently from af50202 to c0ddc7e Compare November 5, 2018 15:39
@hypest
Copy link
Contributor

hypest commented Nov 5, 2018

Had a sneak peak in this PR and it seems that good things are coming, thanks @Tug !

Got to say though that it will be a bit hard to review it from our small mobile side experience so far so, please make an effort to add a PR description when this is ready for review that explains what components are added, short mention of their role and use and so on. Thanks!

@Tug
Copy link
Contributor Author

Tug commented Nov 13, 2018

Closing in favor of #226

@Tug Tug closed this Nov 13, 2018
@Tug Tug deleted the try/experiment-state branch November 13, 2018 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] Not Ready For Review [Type] Enhancement Improves a current area of the editor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants