next-learn
next-learn copied to clipboard
Dependency change to smoothen deployment
#1023
Multiple users include my friend were facing an issue regarding deployment with bcrypt,
I think since this is a tutorial and only for learning purposes using bcryptjs for now is fine, since it is stable and does not cause an error.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| next-learn-starter | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Mar 15, 2025 8:23am |
| next-seo-starter | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Mar 15, 2025 8:23am |
@Priyansh4444 is attempting to deploy a commit to the Vercel Team on Vercel.
A member of the Team first needs to authorize it.
New dependencies detected. Learn more about Socket for GitHub ↗︎
| Package | New capabilities | Transitives | Size | Publisher |
|---|---|---|---|---|
| npm/[email protected] | None | 0 |
112 kB | dcode |
I and other individuals have tried this approach and it does build on vercel but the resulting site's login process fails with a 404 error. Fixing the core issue (building the required packages....bcrypt and sharp) is the better solution than switching to bcryptjs.
There should really be a step in the tutorial that prompts the user to run the approve-builds pnpm command as suggested.
@Priyansh4444 Did you test the login on vercel after deployment to confirm that it works?
PR #1016 already addresses this fix.
Admittedly, I only tested this via localhost on my friends laptop that worked and glad the issue was resolved, this pr can be closed but do agree with confirmation before deployment!