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

[sitecore-jss-nextjs]: Recommending fix to minimized React 18 warnings with nextjs & defaultProps errors #1936

Open
wants to merge 4 commits into
base: release/21.0.0
Choose a base branch
from

Conversation

sengert-alliance
Copy link

@sengert-alliance sengert-alliance commented Sep 27, 2024

…efaultProps errors

Description / Motivation

Field Components have been updated to solve issue with react 18 throwing warnings that defaultProps is deprecated.

This change is to update fields to alleviate defaultProps warnings caused by react 18 which was introduced in jss version 21.7.0 where nextjs was upgraded to next 14.

Testing Details

  • Unit Test Added
  • Manual Test/Other (Please elaborate)
    Ran existing tests to make sure that nothing broke

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@sengert-alliance sengert-alliance changed the title BUGFIX: Recommending fix to minimized React 18 errors with nextjs & d… [sitecore-jss-nextjs]: Recommending fix to minimized React 18 errors with nextjs & defaultProps errors Oct 1, 2024
@pzi
Copy link
Contributor

pzi commented Oct 2, 2024

Would be nice if the Sitecore team could backport this into v20 release, too, for those that are stuck on it for now :)

alternatively, would you accept PRs to fix v20 versions as well @ambrauer ?

@sengert-alliance sengert-alliance changed the title [sitecore-jss-nextjs]: Recommending fix to minimized React 18 errors with nextjs & defaultProps errors [sitecore-jss-nextjs]: Recommending fix to minimized React 18 warnings with nextjs & defaultProps errors Oct 3, 2024
@sengert-alliance sengert-alliance marked this pull request as ready for review October 4, 2024 16:25
@yavorsk yavorsk added the backlog Issue/PR/discussion is reviewed and added to backlog for the further work label Oct 7, 2024
@sengert-alliance
Copy link
Author

@yavorsk is there any way that this can be expedited?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Issue/PR/discussion is reviewed and added to backlog for the further work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants