Skip to content

Make the title block non required#6885

Merged
pnicolli merged 3 commits intomainfrom
title_block_nonrequired
Mar 27, 2025
Merged

Make the title block non required#6885
pnicolli merged 3 commits intomainfrom
title_block_nonrequired

Conversation

@Narcis2005
Copy link
Copy Markdown
Contributor

@Narcis2005 Narcis2005 commented Mar 27, 2025

Caution

The Volto Team has suspended its review of new pull requests from first-time contributors until the release of Plone 7, which is preliminarily scheduled for the second quarter of 2026.
Read details.



If your pull request closes an open issue, include the exact text below, immediately followed by the issue number. When your pull request gets merged, then that issue will close automatically.

Closes #6792

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 27, 2025

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 6f4d3a6
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/67e54c4ef001a700085c10be

@pnicolli pnicolli requested a review from Copilot March 27, 2025 15:59
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the requirement for the title block, aligning the configuration with the new design where the title is no longer mandatory.

  • Removed the 'title' entry from the required blocks list.
Files not reviewed (1)
  • packages/volto/news/6792.feature: Language not supported

@pnicolli pnicolli merged commit 051e923 into main Mar 27, 2025
70 checks passed
@pnicolli pnicolli deleted the title_block_nonrequired branch March 27, 2025 16:02
@boring-cyborg
Copy link
Copy Markdown

boring-cyborg Bot commented Mar 27, 2025

Congratulations on your first merged pull request in this project! 🎉
Congratulations

Thank you for contributing, we are very proud of you! ❤️

@sneridagh
Copy link
Copy Markdown
Member

@pnicolli should we consider this a breaking? I was tempted to backport it, but I think that it is...

@pnicolli
Copy link
Copy Markdown
Contributor

@sneridagh good question. We are not breaking any functionality or styling imho, nothing changes except that there is an extra destructive button for users. It's basically possibly a breaking change for the client, I know some that could see this as a nice added feature and some other that could see this as "why are you giving my team more options to disrupt things without telling me or giving me time to educate?".
Since it's so easy to change in volto 18 and even previous ones if needed, I would err on the safe side and not backport this.

@sneridagh
Copy link
Copy Markdown
Member

@pnicolli I will rename the news item to breaking then.

sneridagh added a commit that referenced this pull request Apr 1, 2025
sneridagh added a commit that referenced this pull request Apr 8, 2025
* main: (356 commits)
  Update or remove broken links (#6953)
  Only accept images in ImageInput component (#6926)
  Bump vite from 5.4.15 to 5.4.16 (#6932)
  Fetch vocabularies and querystring config in the current context (#6935)
  Remove the build from netlify for @plone/components
  Complete missing german translations and Storybook story for 'ContentsDeleteModal' (#6881)
  Remove duplicate configuration item `html_title` (#6933)
  Forwardport hand-cherrypicking the changes happened in Seven for `@plone/registry` relevant for be reintegrated in alpha (`main`). (#6929)
  Fixed regression on brown bag minor version of `@storybook/addon-webpack5-compiler-babel` (#6930)
  Amend PR #6885 changelog to be breaking
  s/Seven/Volto on `main` (#6925)
  Release @plone/registry 3.0.0-alpha.1
  Release @plone/types 2.0.0-alpha.0
  Remove news from packages that won't be released from main
  Fixed merge public folder from addons regression (#6919)
  Fixed handling of errors that are not coming from the backend (#6903)
  Move _hardware-requirements.md include from documentation to volto (#6918)
  Move includes from `plone/documentation` to `plone/volto` (#6917)
  Fix typo in Appextras documentation (#6083)
  Fix gray placeholder on horizontal scroll (#6877)
  ...
sneridagh added a commit that referenced this pull request Apr 9, 2025
* main: (59 commits)
  Update or remove broken links (#6953)
  Only accept images in ImageInput component (#6926)
  Bump vite from 5.4.15 to 5.4.16 (#6932)
  Fetch vocabularies and querystring config in the current context (#6935)
  Remove the build from netlify for @plone/components
  Complete missing german translations and Storybook story for 'ContentsDeleteModal' (#6881)
  Remove duplicate configuration item `html_title` (#6933)
  Forwardport hand-cherrypicking the changes happened in Seven for `@plone/registry` relevant for be reintegrated in alpha (`main`). (#6929)
  Fixed regression on brown bag minor version of `@storybook/addon-webpack5-compiler-babel` (#6930)
  Amend PR #6885 changelog to be breaking
  s/Seven/Volto on `main` (#6925)
  Release @plone/registry 3.0.0-alpha.1
  Release @plone/types 2.0.0-alpha.0
  Remove news from packages that won't be released from main
  Fixed merge public folder from addons regression (#6919)
  Fixed handling of errors that are not coming from the backend (#6903)
  Move _hardware-requirements.md include from documentation to volto (#6918)
  Move includes from `plone/documentation` to `plone/volto` (#6917)
  Fix typo in Appextras documentation (#6083)
  Fix gray placeholder on horizontal scroll (#6877)
  ...
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.

Remove title from required blocks

4 participants