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

MNT Update test label #1343

Merged

Conversation

emteknetnz
Copy link
Member

Copy link
Member

@GuySartorelli GuySartorelli left a comment

Choose a reason for hiding this comment

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

The following still need to be updated:

  • a comment in client/src/containers/Gallery/Gallery.js that mentions componentWillReceiveProps
  • another test client/src/components/UploadField/tests/UploadField-test.js that mentions componentWillReceiveProps

@emteknetnz emteknetnz force-pushed the pulls/2.0/lifecycle branch from e97a923 to 6138411 Compare March 29, 2023 21:52
@emteknetnz
Copy link
Member Author

Updated

I couldn't figure out what comment in Gallery.js referred to even after looking at the CMS 4 version so I just deleted it

@GuySartorelli
Copy link
Member

Failing tests are unrelated

@GuySartorelli GuySartorelli merged commit e0785a8 into silverstripe:2.0 Mar 30, 2023
@GuySartorelli GuySartorelli deleted the pulls/2.0/lifecycle branch March 30, 2023 21:37
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