Skip to content

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

@Adam-it Adam-it self-assigned this Mar 1, 2025
Copy link
Member

@Adam-it Adam-it left a comment

Choose a reason for hiding this comment

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

@NishkalankBezawada Awesome work 👏👏👏
Managed to build it locally without issue
You Rock 🤩

@Adam-it Adam-it merged commit c371d3a into pnp:main Mar 1, 2025
7 checks passed
@LarsThomasNilsson
Copy link

LarsThomasNilsson commented Mar 1, 2025 via email

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.

3 participants