added moenage stuff #1787
added moenage stuff #1787
Conversation
| final _errorMsg = "MoEngage module is not enabled. Please review the Ensemble documentation."; | ||
|
|
||
| MoEngageModuleStub() { | ||
| throw RuntimeError(_errorMsg); |
There was a problem hiding this comment.
Stubs initialise shouldn't be throwing module is not enabled error. This will run in every non moengage apps
| @@ -138,12 +154,15 @@ class NotificationManager { | |||
| } on Exception catch (e) { | |||
| print("NotificationManager: Error receiving notification: $e"); | |||
| } | |||
| print("----------"); | |||
There was a problem hiding this comment.
either convert it to debugPrint or remove it
| @@ -141,6 +141,7 @@ dependencies: | |||
| flutter_slidable: ^3.1.1 | |||
| accordion: ^2.6.0 | |||
| session_storage: ^0.0.1 | |||
| moengage_flutter: ^8.0.0 | |||
There was a problem hiding this comment.
make sure when you make a build for both android and IOS to test it does not need any app entitlement
There was a problem hiding this comment.
-
For iOS, have texted you about testing.
-
For android, I am not sure, wot entitlement is mentioned here, but it you are talking about configurations, while making build we have to update the following files for android for which I have created script:
ensemble.config: for adding moenagageAppID- app level gradle file to include moenageSDK implementation
- Manifest file
MainActivity.ktand also add 2 new.ktfiles for handling notifications callback functions.
|
Have Also removed |
ticket: #1773
Right now creating script for Build system for all configurations related to updating gradle, manifest and kotlin files needed for moenage. Also for iOS.
Wot is implemented for Moenagage is: