Skip to content

Update master #1

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

Merged
merged 94 commits into from
Jul 26, 2019
Merged

Update master #1

merged 94 commits into from
Jul 26, 2019

Conversation

otopba
Copy link
Owner

@otopba otopba commented Jul 26, 2019

No description provided.

Emmanuel Garcia and others added 30 commits July 7, 2019 18:25
…eturned map (#1799)

* make sure the keys are actually added to the returned map

* Add testing, removed a print
* Add a test for rapidly updating in quick succession

* Teach Transaction how to await automatically
…read on Android (#1812)

* Move calls to UI thread

* Update packages/firebase_database/CHANGELOG.md

Co-Authored-By: Chris Yang <[email protected]>
…lt in incorrect thread (#1756)

* [firebase_messaging] fix crash when calling func deleteInstanceID (#1756)
…ion (#1815)

* Add the result section for successfully installations

This section will give feedback on when the installation goes fine, plus informing user that `firebase` takes a while to display the collected crashes into the Firebase Console, since the time is quite long (almost a day) i think it's better to give a feedback here.
* Documentation updates

CHANGELOG
* Remove explicit typing in `_subscription` callback
* Remove `const` modifier for `_kIds` list
* Fix not found ids check typos
* Migrate loop iterator from `:` to `in`
* Fix typo from #1798
* Initial firebase_messaging integration tests
* Change signature of subscribe/unsubscribe
* Wire up the notifications on the native side
* Update unit tests
* Skip the tests now that they don’t actually pass
```dart
const Set<String> _kIds = <String>['product1', 'product2'].toSet();
```

That is not a valid assignment. Hence, I fixed it and on the way I also updated all the links in the README that either did not work at all or did not work from the [pub page](https://pub.dev/packages/in_app_purchase#-readme-tab-).
…stack (#1831)

* Handle case where no class in dart and android (ios is fine)
amirh and others added 29 commits July 16, 2019 10:32
* - Fixes action forwarding by removing extra Activity.
- Report shortcut usage to Android framework.
- Improves example.
This also fetches master that will populate `FETCH_HEAD`

See #1864
* Fix refresh param on getIdToken iOS.

* Update authors.
…with Dart code obfuscation (#1874)

* Fix invokeMethod formatting that caused a bug with Dart code obfuscation
* add plugin for firebase inappmessaging
…1875)

* add firebase_in_app_messaging in README

* Update FlutterFire.md

* Update CODEOWNERS
Needed to unregister the stream handler when disposing of the camera

Per the flutter documentation, the stream handler should be unregistered using setStreamHandler with nil.
https://api.flutter.dev/objcdoc/Classes/FlutterEventChannel.html#/c:objc(cs)FlutterEventChannel(im)setStreamHandler:
This pacakge should be removed entirely soon, so remove it from CI so we
don't spend further time on analyzer failures.
…nc links and launch links (#1687)

* [firebase_dynamic_links] Changed architecture to be able to difference between the dynamic link which opened the app and links clicked on app active or in background.

* [firebase_dynamic_links] Improvements provided by @bparrishMines

* [firebase_dynamic_links] Renaming configure to onLink

* [firebase_dynamic_links] Renaming getLaunchLink to getInitialLink
…on Android (#1891)

* add support to access android filesDir
…arnings (#1884)

* Fix Android compiler warnings

* Fix deprecation warnings for location plugin

* Specify min Flutter SDK
* update AUTHORS file

* add new method recordError

* rename onError to recordFlutterError

Co-Authored-By: Collin Jackson <[email protected]>
@otopba otopba merged commit 073fca5 into otopba:master Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.