Skip to content

fix: re-pin @vercel/nft#6610

Merged
serhalp merged 1 commit intomainfrom
fix/nft-pin
Aug 8, 2025
Merged

fix: re-pin @vercel/nft#6610
serhalp merged 1 commit intomainfrom
fix/nft-pin

Conversation

@eduardoboucas
Copy link
Member

@eduardoboucas eduardoboucas commented Aug 8, 2025

Summary

Reverts #6573, which seems to cause this issue.

Reverting until we can find the root cause.

@eduardoboucas eduardoboucas requested a review from a team as a code owner August 8, 2025 14:49
@serhalp serhalp enabled auto-merge (squash) August 8, 2025 15:05
@serhalp serhalp merged commit 559b04f into main Aug 8, 2025
66 of 67 checks passed
@serhalp serhalp deleted the fix/nft-pin branch August 8, 2025 15:05
serhalp added a commit that referenced this pull request Dec 4, 2025
We had [previously pinned to 0.29.4](#6610) because [0.30 introduced a bug](https://answers.netlify.com/t/react-router-app-crashing-because-it-cant-find-package/155391).

This was fixed in vercel/nft#550, released in 1.1.0.

I've confirmed this:

1. Clone https://github.com/netlify/react-router-template/
2. `npm i`
4. ✅ `rm -rf .netlify/ && npx -y netlify@latest deploy --skip-functions-cache`
5. ✅ `rm -rf .netlify/ && npx -y netlify@23.0.0 deploy --skip-functions-cache`
6. ✅ `rm -rf .netlify/ && npx -y netlify@23.1.3 deploy --skip-functions-cache`
7. ❌ `rm -rf .netlify/ && npx -y netlify@23.1.2 deploy --skip-functions-cache`
8. Clone this branch, `npm i && npm run build`. Clone netlify-cli, `npm link ../build/packages/zip-it-and-ship-it`.
9. ✅ `rm -rf .netlify/ && ../cli/bin/run.js deploy --skip-functions-cache`
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