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

Refactored SelectMetadataField.jsx to functional component #5570

Merged
merged 9 commits into from
Sep 5, 2024

Conversation

lorstenoplo
Copy link
Contributor

Ref: #4460

I have converted the src/components/manage/Blocks/Search/widgets/SelectMetadataField.jsx to a functional component and removed unused code.

Copy link

netlify bot commented Dec 24, 2023

Deploy Preview for volto ready!

Name Link
🔨 Latest commit 045bd52
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/65a02ad0d01f4c0008fad334
😎 Deploy Preview https://deploy-preview-5570--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 configuration.

Copy link

netlify bot commented Dec 24, 2023

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 2fb7e6e
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/66a21e4e06a03e0008855e9d

@lorstenoplo lorstenoplo changed the title Changed SelectMetadataField.jsx to functional component Refactored SelectMetadataField.jsx to functional component Dec 24, 2023
.gitpod.yml Outdated Show resolved Hide resolved
packages/volto/news/5570.internal Outdated Show resolved Hide resolved
@plone plone deleted a comment from dhruvgupta0503 Jan 11, 2024
@stevepiercy
Copy link
Collaborator

@lorstenoplo please see Item 2 in https://6.docs.plone.org/contributing/first-time.html#create-a-pull-request-from-your-fork and edit your pull request description.

@stevepiercy
Copy link
Collaborator

Final step, @lorstenoplo please see Item 2 in https://6.docs.plone.org/contributing/first-time.html#create-a-pull-request-from-your-fork and edit your pull request description. Then it can be reviewed.

Copy link
Collaborator

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

I triaged this PR for review by a core developer. It's ready.

@lorstenoplo
Copy link
Contributor Author

Why is the pr not being merged?

@stevepiercy
Copy link
Collaborator

@plone/volto-team this PR needs review.

@tiberiuichim tiberiuichim self-requested a review March 2, 2024 09:49
@davisagli davisagli merged commit 8c4c156 into plone:main Sep 5, 2024
44 checks passed
sneridagh added a commit that referenced this pull request Sep 14, 2024
* main: (97 commits)
  Release 18.0.0-alpha.43
  Release @plone/slate 18.0.0-alpha.18
  Release @plone/client 1.0.0-alpha.17
  Added a notice in the upgrade guide for CookiePlone as the recommended way for generating projects and add-on boilerplate (#6286)
  Fix typo in Changelog for #6285 (#6287)
  Update to Volto 6.0.13 (#6285)
  Fix order component (#6284)
  Remove dangling `.only` in Cypress test (#6282)
  Empty GridBlocks will not be rendered in 'View' (#6280)
  Refactor Content upload modal  (#5047)
  Passes the `user` to the `restricted` function of the block settings (#6271)
  Refactored SelectMetadataField.jsx to functional component (#5570)
  Remove "Generate tabs for items other than folders" field from navigation control panel (#6278)
  Refactor DatetimeWidget (#6213)
  Added make i18n to volto (#6274)
  Place the actions-storybook in correct folder (#6275)
  Refactor ReferenceWidget (#5093)
  Fix error in SortOn component when there is no sort selected. (#6273)
  When user changes location, set the userSession.token value based on cookie (#6079)
  Trivial MyST syntax enhancement
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants