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

fix reordering of slots function #5840

Merged
merged 7 commits into from Mar 19, 2024
Merged

fix reordering of slots function #5840

merged 7 commits into from Mar 19, 2024

Conversation

steffenri
Copy link
Contributor

No description provided.

Copy link

netlify bot commented Mar 5, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 385081e
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/65e9f4143fcdef0008686d5e

Copy link

netlify bot commented Mar 5, 2024

Deploy Preview for volto canceled.

Name Link
🔨 Latest commit 385081e
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/65e9f4148b55290008fde7a2

@sneridagh
Copy link
Member

@steffenri that would need a test... and understand why the existing ones pass :)

@steffenri steffenri marked this pull request as ready for review March 7, 2024 16:57
@steffenri
Copy link
Contributor Author

@sneridagh Added more tests for all the use cases to show what the problem is. Also fixed the cause of the problem. There might be a better way to do this though.

@steffenri steffenri requested a review from sneridagh March 7, 2024 16:58
@sneridagh
Copy link
Member

@davisagli do you know if we can simplify this?

@sneridagh sneridagh merged commit 482bced into main Mar 19, 2024
73 checks passed
@sneridagh sneridagh deleted the fix-slot-reordering branch March 19, 2024 11:48
sneridagh added a commit that referenced this pull request Apr 16, 2024
* main: (21 commits)
  Use fieldset `id` instead of the fieldset `title` in Form when rendering the Field component (#5921)
  Fix registry wrong default primitive type (#5925)
  Release @plone/types 1.0.0-alpha.8
  Improve types for APIExpanders, export all (#5918)
  Missing step in storybook migration (#5913)
  Push missing .d.ts files
  Release 18.0.0-alpha.25
  Release generate-volto 9.0.0-alpha.14
  Update Docs: Pastanaga UI and Quanta (#5903)
  Upgrade to StoryBook 8 (#5912)
  Fix Storybook in app gen for Volto 18 (#5911)
  Release 18.0.0-alpha.24
  Upgrade `@typescript-eslint` (#5910)
  Add new cypress helper for slate `getSlateEditorAndType` (#5909)
  Add id attributes to discussions container and individual comments (#5905)
  Move testing-library cypress commands import to inner commands so it can be imported (#5906)
  Release 18.0.0-alpha.23
  Release @plone/registry 1.5.4
  Improve the usage of RAZZLE_JEST_CONFIG (#5901)
  fix reordering of slots function (#5840)
  ...
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.

None yet

2 participants