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

Update to WordPress 5.5 #239

Closed
rmccue opened this issue Jul 16, 2020 · 7 comments
Closed

Update to WordPress 5.5 #239

rmccue opened this issue Jul 16, 2020 · 7 comments
Assignees

Comments

@rmccue
Copy link
Member

rmccue commented Jul 16, 2020

Integration issue for WordPress 5.5, which is planned for August.

Notable things we need to account for:

Likely more to note too.

@rmccue rmccue added this to the 5.0 milestone Jul 16, 2020
@rmccue
Copy link
Member Author

rmccue commented Jul 20, 2020

Some raw Gutenberg screenshots from updating with no further changes:

Screenshot_2020-07-20 Edit Post ‹ Altis Demo — Altis(3)

Screenshot_2020-07-20 Edit Post ‹ Altis Demo — Altis(4)

Additionally, the mobile and tablet "preview" views are completely broken for me, showing a contentless preview about 20px tall, unsure if that's from our code or not (didn't get a screenshot).

Looks like our colour scheme will need updating, and we need to integrate with the new Gutenberg toolbar styling. The experiments and publication checklist features look fine and shouldn't need changes.

IMO, the Gutenberg toolbar styling looks terrible, and it's also very jarring as a user to see it change. It may be worth considering whether we can revert the visual changes altogether.

@rmccue
Copy link
Member Author

rmccue commented Jul 20, 2020

Related, Gutenberg changelog for 5.5: WordPress/gutenberg#23669

@lachlanj
Copy link

It may be worth considering whether we can revert the visual changes altogether.

I'm not sure if this is a serious consideration or just a frustration with the new changes but I would hope that we don't try to revert this. WordPress has changed it's UI many times over the years for better or worse, but personally I think mostly for the better. It would be a shame to have to deviate from this.

@roborourke
Copy link
Contributor

roborourke commented Sep 1, 2020

A REST API update is showing these warnings:

register_rest_route was called incorrectly. The REST API route definition for ... is missing the required permission_callback argument. For REST API routes that are intended to be public, use __return_true as the permission callback. Please see Debugging in WordPress for more information. (This message was added in version 5.5.0.)

This affects the following routes:

  • workflows/v1/webhooks/(?P[\w\-\.]+)\/(?P[\w\-\.]+)
  • workflows/v1/notifications/(?P[\d]+)/(?P[\d]+)
  • workflows/v1/notifications/(?P[\d]+)
  • elasticpress/v1/pointer_search
  • elasticpress/v1/pointer_preview

@roborourke
Copy link
Contributor

PR opened for Workflows, we may have to workaround the ElasticPress endpoint registry for now as I'm not sure when 3.5.0 which has the fix will ship.

@roborourke
Copy link
Contributor

roborourke commented Sep 1, 2020

I looked into image editing, in short it sends rotation and crop data to the endpoint /wp/v2/media/:id/edit which handles the transformation and sends back a new attachment. While we could potentially hijack that endpoint and force it to return the same attachment with a modified URL containing the tachyon args there are 2 things standing in the way of that:

As a result I don't believe this is something we can do super quickly for v5 because it's an infrastructure roll out and tricky media modal work.

@roborourke
Copy link
Contributor

Marking this as done, the gaussholder one can come later is indirectly related.

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

No branches or pull requests

3 participants