refactor(BIG): renamed dart_firebase_admin to firebase_admin_sdk#217
refactor(BIG): renamed dart_firebase_admin to firebase_admin_sdk#217
Conversation
Fixes #209 Added notes about the history of the package Updated links to Invertase bits that needed to be cleanedup. Refactored the changelogs Updated the docs
There was a problem hiding this comment.
Code Review
This pull request renames the package from dart_firebase_admin to firebase_admin_sdk, involving extensive updates to documentation, directory structures, and import statements across the repository. The review feedback identifies that the updated CHANGELOG.md is missing the release notes for version 0.5.0 and points out an incorrect file path in a comment within the functions example.
Coverage Report✅ Coverage 74.66% meets 40% threshold Total Coverage: 74.66% Package Breakdown
Minimum threshold: 40% |
demolaf
left a comment
There was a problem hiding this comment.
there's a broken symlink for README.md in the root of the project — it still points to packages/dart_firebase_admin/README.md instead of packages/firebase_admin_sdk/README.md.
|
@kevmoo I've opened a PR #218 to fix the symlink issue and also a guide on how to use authentication via OAuth 2.0 refresh token to match the firebase docs https://firebase.google.com/docs/admin/setup. |
|
Follow-up here: firebase/firebase-functions-dart#123 |
Fixes #209
Added notes about the history of the package
Updated links to Invertase bits that needed to be cleanedup.
Refactored the changelogs
Updated the docs