next-firebase-course icon indicating copy to clipboard operation
next-firebase-course copied to clipboard

Firebase v10

Open SkylerA opened this issue 1 year ago • 1 comments

This PR seeks to update the firebase syntax from v8 to v10 and provide a reference of necessary changes for anyone building the project with @latest packages.

Initially, I tired to only update the minimum dependencies of firebase, react and react-dom, but at the end I ran into one issue with getUserWithUsername throwing an error about collection() not being passed a valid reference even though it was being passed firestore and was seemingly working fine in other files. In the end, I decided to just update react/next.js to latest and this resolved the issue.

The following changes are made:

  • Update necessary packages to latest
  • Fix bug with unheart showing because exists was being treated as a boolean instead of a function Similar PR
  • Copies react-hook-form updates from https://github.com/fireship-io/next-firebase-course/pull/4 (If updating this course, please integrate the original authors PR first so they are credited)
  • Temp fixes <Link> containing <a> errors with legacyBehavior. CSS might need an update after removing <a> and legacyBehavior. While writing this PR note, i just realized i blindly added legacyBehavior to all links while getting ready to check in. This is actually only needed on links that contain <a>
  • Updates all firebase commands and adds a few TODOs to consider

Note This is my first time using firebase so there might be a better implementation. I've placed a few TODOs with points you might need to consider if using this project as a boilerplate.

SkylerA avatar Jul 30 '23 18:07 SkylerA

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
next-firebase-course ❌ Failed (Inspect) Jul 30, 2023 6:25pm

vercel[bot] avatar Jul 30 '23 18:07 vercel[bot]