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 Display.jsx. #6204

Closed
wants to merge 3 commits into from
Closed

Conversation

FreakForever
Copy link

Refactored Display.jsx from class-based component to functional-based component.

Copy link

netlify bot commented Jul 27, 2024

Deploy Preview for plone-components canceled.

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

@stevepiercy
Copy link
Collaborator

@FreakForever one of the CI checks failed. I restarted it, as it could be a flaky test. What results did you get from running acceptance tests locally, as described in Code quality?

@stevepiercy
Copy link
Collaborator

It looks like this may be a duplicate of #5066. @FreakForever please read Item 2 Things not to do.

The last remaining CI check passed.

@plone/volto-team this PR is ready for review, if #5066 does not get merged first.

@FreakForever
Copy link
Author

Hey @stevepiercy will this pr get merged ? as in the pr mentioned #5606 some tests are failing.

@stevepiercy
Copy link
Collaborator

@FreakForever please read Expectations of first-time contributors, especially the warning.

We are all volunteers with limited time, and this issue is very low priority.

If this is for a class assignment, then please have your instructor contact me directly. It is unfair to you and volunteers for an instructor to raise unrealistic expectations or make demands.

@FreakForever
Copy link
Author

FreakForever commented Jul 30, 2024 via email

@davisagli
Copy link
Member

Closing as a duplicate of #5066. Thanks for your contribution.

@davisagli davisagli closed this Aug 10, 2024
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.

5 participants