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

Plugin updates: Gutenberg 7.7.1 #40078

Closed
19 of 23 tasks
ramonjd opened this issue Mar 11, 2020 · 6 comments
Closed
19 of 23 tasks

Plugin updates: Gutenberg 7.7.1 #40078

ramonjd opened this issue Mar 11, 2020 · 6 comments
Labels
[Goal] Gutenberg Working towards full integration with Gutenberg [Type] Janitorial

Comments

@ramonjd
Copy link
Member

ramonjd commented Mar 11, 2020

Gutenberg 7.7.1 was released on March 13, 2020.

This issue tracks progress upgrading wpcom.

With 7.7.1, we're skipping stable versions (7.4.0 (potentially), 7.5.0 and 7.6.0).

7.4.0 is currently waiting in the halls until the ongoing work on the data migration project (pbAok1-f8-p2) is complete, however the completion date is unknown with estimation 5-6 days, starting on Monday March 16th.

Gutenberg

  • Install stable version: D40297-code
  • Activate stable version on edge sites: D40215-code
  • Activate stable version on prod: D40794-code
  • Cleanup old versions of GB (D39409-code)
  • Request update for AT sites (p9o2xV-N2-p2)

Other

Bugs encountered 🐞

Punted after release

@ramonjd ramonjd added [Goal] Gutenberg Working towards full integration with Gutenberg [Type] Janitorial labels Mar 11, 2020
@ramonjd
Copy link
Member Author

ramonjd commented Mar 12, 2020

@p-jackson pointed out the changes to the header in 7.7

Current

Screen Shot 2020-03-12 at 3 44 43 pm

With 7.7 enabled

Screen Shot 2020-03-12 at 3 45 31 pm

This breaks our back button pretty hard.

My uneducated guess is that we'd need to update the FSE BackButtonOverride hook method ?

Also filed as an issue here: #40065

@ramonjd ramonjd changed the title Plugin updates: Gutenberg 7.7.0 Plugin updates: Gutenberg 7.7.1 Mar 13, 2020
@noahtallen
Copy link
Contributor

My uneducated guess is that we'd need to update the FSE BackButtonOverride hook method ?

That override will only work for dotcom-fse sites, so I think it will need fixed elsewhere too... Trying to remember where that is 🤔

This also breaks the frame around the editor for dotcom-fse:

Screen Shot 2020-03-12 at 5 46 50 PM

And the new "document sidebar" button is hidden by our staging icon

Screen Shot 2020-03-12 at 5 48 11 PM

@noahtallen
Copy link
Contributor

In dotcom-fse, we should probably just stick with the overridden back button and remove the W icon for now. I say this because the W icon in the template part editor won't make sense -- we still need to get back to the page editor, not back to wp-admin. And we should probably keep the back button in the page editor consistent with that for now.

@roo2
Copy link
Contributor

roo2 commented Mar 22, 2020

hmmm so what's the best release process for getting neccessary fixes changes to gutenberg into this release? ( I'm thinking of this one #40127)

This is the process as I understand it:

Gutenberg PR -> Gutenberg Master -> Tag a new Gutenberg release v7.7.x -> zip file -> install on wpcom edge sites -> test -> call for testing -> make it live for everyone

The problem then is that the new Gutenberg release v7.7.x will include a whole set of commits that haven't been tested with wpcom so we almost have to restart the testing effort rather than just checking this bug off the list of known issues.

But we can also manually change the zip file right, it sounds like that's the safer solution in this case, is that what you guys were thinking?

Currently v7.7.0 is a tag on the master branch, I've seen teams create release branches for this kind of complex release and then apply or cherry-pick commits on to that branch.. but that would be a different future discussion I think.

@ramonjd
Copy link
Member Author

ramonjd commented Mar 23, 2020

hmmm so what's the best release process for getting neccessary fixes changes to gutenberg into this release?

For major blockers we can update the files ourselves on .com. The only headache will be making sure we file a corresponding PR on the WordPress/Gutenberg repo.

The problem then is that the new Gutenberg release v7.7.x will include a whole set of commits that haven't been tested with wpcom so we almost have to restart the testing effort rather than just checking this bug off the list of known issues.

I think we could carry on as we've been doing: deal with each release as it comes out from Gutenberg, and fixing things on our end + creating Gutenberg PRs to ensure that our changes make it into future versions.

ramonjd added a commit that referenced this issue Mar 25, 2020
…obile and desktop because webdriver can't close the page template selector.

We're temporarily disabling this test until we smooth out the integration of Gutenberg 7.7.1 See: #40078
@simison
Copy link
Member

simison commented Mar 26, 2020

I'll close this — we can continue in 7.8.0 issue for the next version.

@simison simison closed this as completed Mar 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Goal] Gutenberg Working towards full integration with Gutenberg [Type] Janitorial
Projects
None yet
Development

No branches or pull requests

4 participants