-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: resolve several issues #21
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the firebase instance should be in the constructor of the notifications service
...ges/flutter_notification_center_firebase/lib/src/services/firebase_notification_service.dart
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also remove the test folder in the root and the pubspec.yaml. You could add a melos.yaml from another component if you want to indicate that it is a monorepo
c838094
to
90df385
Compare
f0406e6
to
7d9ac45
Compare
d253e50
to
8e93b5d
Compare
5ab883e
to
194449e
Compare
Closes
#8
#10
#15
#19