[MOB-6055] - [OMNI CG] - Add callbacks to setting email/user id #570
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[MOB-6055] callback on setEmail/setUserId
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #570 +/- ##
==========================================
- Coverage 61.99% 61.93% -0.07%
==========================================
Files 68 68
Lines 3984 3998 +14
Branches 465 466 +1
==========================================
+ Hits 2470 2476 +6
- Misses 1265 1272 +7
- Partials 249 250 +1
☔ View full report in Codecov by Sentry. |
davidtruong
approved these changes
May 17, 2023
Ayyanchira
changed the title
Ayyanchira
changed the title
evantk91
added a commit
that referenced
this pull request
Sep 7, 2023
* sets custom sound at notification channel * updates notification channel builder to set sound * minor edits * sets sound at notification channel * minor edits * sets up channel id and channel name for each unique sound * adds conditionals for default channel creation * investigating edge cases * debugging * adds default sounds for edge cases * adds default channel id and channel name for edge cases * minor edits * minor edits * cleans up conditionals * adds import for IterableNotificationTest * reorganizes and checks resources folder for matching sound file * minor edits * minor edits * minor edits * updates channel names to sound names * resolves null pointer exception for soundName * removes unneccessary package from build.gradle * README updates * [MOB-5532] - Adding remote notification sound Also removing some checks which seem removable. Still WIP * ChannelId was getting set to null when no sound * Remove all other channels which are not active Remove all the other channels which are not actively showing notification on the notification tray. Warning: Have to check if it only removes the notification created by iterable package? or the entire app's other channels as well. We might have to add iterable_ or some kind of identifier to delete only Iterable ones. * Checkstyle fix * adds resource check for existing local sound and updates soundName checks * adds soundId and soundUrl checks for channel id and channel name * passes soundId down to getSoundUri * Revert "passes soundId down to getSoundUri" This reverts commit 0d2e825. * adds null check to getSoundUri * checkstyle fix * puts sound uri check in createNotificationChannel * reorganizes to do the Uri logic in getSoundUri * checkstyle fixes * addresses comments * reverts back to not deleting inactive channels * Update iterableapi/src/main/java/com/iterable/iterableapi/IterableNotificationHelper.java Co-authored-by: Akshay Ayyanchira <[email protected]> * addresses code climate fixes * [MOB-5664] - Revert channel name implementation Revert the code where the channel name was derived by reading value from Android Manifest. If not value is found, a default name "iterable channel" is provided. Now, with custom sound, if a soundURI exists from a provided custom sound, only then soundName is treated as a channel name. Otherwise, old way of handling the name still persists. * [MOB - 5694] - Add try catch around animation Adding try catch around show and hide webview animation * Log warn instead of error checkSDKInitialization logs error message. Log warning message instead * Add firetv/ott support to getmessages call * For firetv call update user with device details upon initialization * Set DEVICE_NOTIFICATIONS_ENABLED key only when registering for push * Fix firetv object structure * Review fixes * [MOB - 5873] - Public getters for Email UserID and Auth _email, _userID and _authToken to accessible by developers. :) * [MOB - 5874] - Schedule auth refresh when auth null Schedule a 10 second auto refresh for auth handler if auth token is found null when initializing. Also, when authHandler receives a authToken from handler in its handler, a similar null check will schedule 10 second refresh throwing a onTokenRegistrationFailure. This callback should allow them to refresh authToken. * create issue in jira when github issue opened * update secret names * add private constructor to prevent instance creation * update unit test * Added import for DeviceInfoUtils * Merge pull request #474 from Iterable/jay/MOB-4666-in-apps-config-memory [MOB-4666] in apps config memory * CI fix Attempt to fix this error * What went wrong: Execution failed for task ':app:processDebugAndroidTestManifest'. > Could not resolve all files for configuration ':app:debugAndroidTestRuntimeClasspath'. > Could not download kappuccino-1.2.1.aar (br.com.concretesolutions:kappuccino:1.2.1) > Could not get resource 'https://jcenter.bintray.com/br/com/concretesolutions/kappuccino/1.2.1/kappuccino-1.2.1.aar'. > Could not GET 'https://jcenter.bintray.com/br/com/concretesolutions/kappuccino/1.2.1/kappuccino-1.2.1.aar'. > Connection reset * [MOB-6055] - [OMNI CG] - Add callbacks to setting email/user id (#570) * Change for supporting callback feature on setEmail/setUserId * fixed failuire callback * add tests for setEmail and setUserId callbacks * changed to iterablehelper callbacks * update test with iterablehelper callbacks * Update IterableApi.java * Retaining the previous method signatures --------- Co-authored-by: Hardik Mashru <[email protected]> Co-authored-by: “Akshay <“[email protected]”> * [MOB 5730] Add callbacks to reading/removing in-app messages (#557) (#583) * [MOB 5730] Add callbacks to reading/removing in-app messages (#557) * add callback for setRead/removeMessage * modify test for setRead and added test for removeMessage * fixes * Update build.gradle * removed resultcallbackhandler * Update IterableInAppManager.java * fixes --------- Co-authored-by: Akshay Ayyanchira <[email protected]> Co-authored-by: Hardik Mashru <[email protected]> * Fixing and adding test method --------- Co-authored-by: devcsomnicg <[email protected]> Co-authored-by: Hardik Mashru <[email protected]> Co-authored-by: “Akshay <“[email protected]”> * [MOB - 6493] - Message read and remove bug fix (#592) Co-authored-by: “Akshay <“[email protected]”> * MOB-5132: Fix deep link issue after app is opened from notification (#546) (#593) Co-authored-by: devcsomnicg <[email protected]> * [MOB-6309] prepares EUDC updates for release (#572) * stashed changes * adds data center to config and associated unit tests * adds excluding kotlin files to javadoc check * moves IterableDataRegion to IterableConstants.java * gets rid of extra lines * removes jacoco.exe * adds endpoint override to IterableApi * removes logging statement * sets up base url at IterableRequestTask * minor edits * refactors to pull endpoint directly from config value * removes unfinished unit test * removes jacoco.exec * removes white space --------- Co-authored-by: [email protected] <[email protected]> * removes extra dependency (#609) Co-authored-by: [email protected] <[email protected]> * MOB-5132 (#613) * MOB-5132: Fix deep link issue after app is opened from notification * Fix deep link for buttons It fixes the issue of deep link on buttons --------- Co-authored-by: Akshay Ayyanchira <[email protected]> * Ensure IterableTrampolineActivity always launched (#553) (#618) - Ensure IterableTrampolineActivity always launched to handle all notifications - Reverts back to launchMode="singleInstance" to ensure that the activity is always launched - Use excludeFromRecents="true" to ensure that the activity is not shown in the recent apps list Co-authored-by: Tim Nortman <[email protected]> --------- Co-authored-by: evangreer <[email protected]> Co-authored-by: Brad Umbaugh <[email protected]> Co-authored-by: “Akshay <“[email protected]”> Co-authored-by: Akshay Ayyanchira <[email protected]> Co-authored-by: amanforindia <[email protected]> Co-authored-by: Ilya Brin <[email protected]> Co-authored-by: Hardik Mashru <[email protected]> Co-authored-by: Ilya Brin <[email protected]> Co-authored-by: devcsomnicg <[email protected]> Co-authored-by: Hardik Mashru <[email protected]> Co-authored-by: [email protected] <[email protected]> Co-authored-by: Tim Nortman <[email protected]>
evantk91
pushed a commit
that referenced
this pull request
Sep 8, 2023
* Change for supporting callback feature on setEmail/setUserId * fixed failuire callback * add tests for setEmail and setUserId callbacks * changed to iterablehelper callbacks * update test with iterablehelper callbacks * Update IterableApi.java * Retaining the previous method signatures --------- Co-authored-by: Hardik Mashru <[email protected]> Co-authored-by: “Akshay <“[email protected]”>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔹 Jira Ticket(s) if any
✏️ Description