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

Make Volto requirements more explicit. #400

Closed
wants to merge 1 commit into from
Closed

Make Volto requirements more explicit. #400

wants to merge 1 commit into from

Conversation

fredvd
Copy link
Contributor

@fredvd fredvd commented Jun 25, 2024

No description provided.

@fredvd fredvd requested a review from sneridagh June 25, 2024 14:02
@sneridagh
Copy link
Member

@fredvd I don't think that we did nothing breaking that would make 4.x not work with Volto 17.

@fredvd
Copy link
Contributor Author

fredvd commented Jun 28, 2024

I tried it in a 17 project by pinning 4.0.0 and it fails. Iirc something with not finding ‘@plone/components’. That’s why I started looking for more info. I can give you the exact error if you want to see it.

@sneridagh
Copy link
Member

@fredvd uh, I see the problem now... I will fix it.

@sneridagh sneridagh closed this in 269d983 Jun 28, 2024
@sneridagh
Copy link
Member

@fredvd now should be fine, can you test it using 4.0.1?
Now the dependencies are ok:

32  "dependencies": {
33    "@plone/components": "2.0.0-alpha.11"
34  },

sneridagh added a commit that referenced this pull request Jun 28, 2024
* main:
  Finish interrupted push
  Fixes #400, install in Volto 17
  Fix Invalid html structure in caption component (#401)
@sneridagh sneridagh reopened this Jun 28, 2024
@sneridagh
Copy link
Member

@sneridagh
Copy link
Member

sneridagh commented Jul 2, 2024

@fredvd I confirmed that using 4.0.1 package it does work with Volto 17.
I am introducing a new requirement though, when merging #405. Volto 17.18.1 at least, so I will issue a new breaking in there to make it noticeable.

@sneridagh sneridagh closed this Jul 2, 2024
sneridagh added a commit that referenced this pull request Jul 2, 2024
* main: (25 commits)
  Upgrade to a39, enable new image widget (#405)
  Finish interrupted push
  Fixes #400, install in Volto 17
  Fix Invalid html structure in caption component (#401)
  build deps
  Upgrade to Volto a37 (#403)
  Release 4.0.0
  Fix release-it script
  Changelog
  Fix tabbing in header (reapplies #346 as breaking) (#374)
  Customizable mobile navigation (#393)
  Fix Description block width in Edit/Add (#394)
  Slight improvements
  Adjust height
  Add BMv3 ready logo
  Remove not needed dockerfiles folder
  Update setup. Use new images. (#390)
  fix Logo alt-Title + add German translations (#337)
  fix link in introduction block being smaller than normal text (#366)
  Release 3.3.2
  ...
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.

2 participants