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

Volto slate upgrade guide cleanup #3529

Merged
merged 2 commits into from
Jul 29, 2022
Merged

Conversation

stevepiercy
Copy link
Collaborator

@stevepiercy stevepiercy commented Jul 28, 2022

This addresses the missed review items from #3508

@netlify
Copy link

netlify bot commented Jul 28, 2022

Deploy Preview for volto ready!

Name Link
🔨 Latest commit e53ac9f
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/62e2ee85d4d75100088aaa75
😎 Deploy Preview https://deploy-preview-3529--volto.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@stevepiercy
Copy link
Collaborator Author

Two unresolved questions from:
#3508

  1. We say that draftJS will be removed in Volto 18, but we do not say to what version of Plone that might correspond. Is there a roadmap for removal of draftJS in a future version of Plone, such as Plone 6.2 or 7.0, or is it "whatever version number of Plone is at whenever we release Volto 18"?
  2. Do we return a deprecation warning to the developer if they use draftJS? I don't know if that is even possible in Volto, but we do that in Pyramid when a deprecated method is called.

@stevepiercy stevepiercy self-assigned this Jul 28, 2022
@stevepiercy stevepiercy added this to the Plone 6.0 milestone Jul 28, 2022
@cypress
Copy link

cypress bot commented Jul 28, 2022



Test summary

353 0 15 0


Run details

Project Volto
Status Passed
Commit e53ac9f
Started Jul 28, 2022 8:19 PM
Ended Jul 28, 2022 8:31 PM
Duration 12:34 💡
OS Linux Ubuntu - 20.04
Browser Chrome 103

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@sneridagh
Copy link
Member

Two unresolved questions from: #3508

  1. We say that draftJS will be removed in Volto 18, but we do not say to what version of Plone that might correspond. Is there a roadmap for removal of draftJS in a future version of Plone, such as Plone 6.2 or 7.0, or is it "whatever version number of Plone is at whenever we release Volto 18"?

More the latter because, we do not know that yet. We will assign Volto releases to Plone versions but we can't tell upfront. I have a plan to write down the Plone's product strategy and discuss it during the conf with the Release team and people interested.

We did this "deprecated but will be removed two majors from now" before, it worked well. Allows people to adjust.

  1. Do we return a deprecation warning to the developer if they use draftJS? I don't know if that is even possible in Volto, but we do that in Pyramid when a deprecated method is called.

We could throw a console message if the configuration is set to draftJS, but never did it.

@sneridagh sneridagh merged commit 5f57b18 into master Jul 29, 2022
@sneridagh sneridagh deleted the volto-slate-upgrade-guide-cleanup branch July 29, 2022 08:16
sneridagh added a commit that referenced this pull request Aug 3, 2022
* master: (36 commits)
  fix propTypes warning for Pagination component (#3543)
  Update documentation on how to write a Slate plugin (#3532)
  Updatebrowserlist (#3526)
  Back to development (@plone/volto-testing)
  Release @plone/volto-testing 2.0.1
  Prepare for release
  Upgrade `cypress-axe` since it was forcing `cypress` < 10 version.
  Back to development (@plone/volto-testing)
  Release @plone/volto-testing 2.0.0
  Upgrade to Cypress 10
  Back to development
  Release 16.0.0-alpha.20
  Prepare for release
  Migrate to Cypress 10 (#3541)
  Fixed Storybook configuration for add-ons (#3357)
  make Search page title translatable (#3538)
  Volto slate upgrade guide cleanup (#3529)
  Back to development
  Release 16.0.0-alpha.19
  Allow stripping elements in the headlines (#3523)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants