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

Upgrade to SPFx v1.20.0 #1455

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

NishkalankBezawada
Copy link
Contributor

Q A
Bug fix? yes - react-application-machine-translations unable to build, due to dependency mismatch
New feature? no
New sample? no
Related issues? fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?

Fixing the dependency mismatch and also upgrading the spfx version.

react-application-machine-translations unable to build, due to dependency mismatch.

Seems like the update of the sample was done few months ago, as below,

image

but, somehow, it was not. Please refer to the package.json as below,

image

the mismatch of the devDependencies with @microsoft/sp-build-web is with version 1.18.0, and rest are on version 1.12.1, which ideally shouldnt be as such.

Hence, Creating this PR to fix this, and also upgrade the SPFx version to 1.20.0

Thanks,
Nish

@NishkalankBezawada NishkalankBezawada changed the title Upgrade to v1.15.0 Upgrade to v1.18.0 Dec 8, 2024
@LarsThomasNilsson
Copy link

Blending work Nishkalank.

@NishkalankBezawada NishkalankBezawada changed the title Upgrade to v1.18.0 Upgrade to v1.19.0 Dec 9, 2024
@NishkalankBezawada NishkalankBezawada changed the title Upgrade to v1.19.0 Upgrade to v1.20.0 Dec 9, 2024
@NishkalankBezawada NishkalankBezawada changed the title Upgrade to v1.20.0 Upgrade to SPFx v1.20.0 Dec 9, 2024
@NishkalankBezawada
Copy link
Contributor Author

Hello @hugoabernier

I have upgraded this solution and have successfully built it. Request you to kindly review this.

Thanks,
Nish

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