-
Notifications
You must be signed in to change notification settings - Fork 4.1k
[firebase_storage] Support Android v2 embedding #1295
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
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.
LGTM
@@ -10,6 +10,7 @@ dependencies: | |||
firebase_core: ^0.4.0 | |||
uuid: ^1.0.0 | |||
http: ^0.12.0 | |||
e2e: ^0.2.1 |
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.
should this be a dev_dependency?
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.
Done!
Reminder to sync and merge this |
Once the build is green, I'll rebase and merge this PR. |
2c50ab4
to
eb6d861
Compare
7c0406f
to
1f5b4d6
Compare
1f5b4d6
to
6154776
Compare
Fixes: flutter/flutter#41877