Skip to content
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

[MOB-6576] merge EUDC changes into encryption branch #600

Merged
merged 5 commits into from
Jul 17, 2023
Merged

Conversation

evantk91
Copy link
Contributor

@evantk91 evantk91 commented Jul 17, 2023

🔹 Jira Ticket(s) if any

✏️ Description

This pull request merges EUDC changes into encryption branch to prepare for release.

Ayyanchira and others added 4 commits June 14, 2023 13:24
…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]”>
* 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]>
@evantk91 evantk91 requested a review from Ayyanchira July 17, 2023 17:26
Copy link
Member

@Ayyanchira Ayyanchira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@evantk91 evantk91 merged commit f7623cd into encryption Jul 17, 2023
4 of 8 checks passed
evantk91 added a commit that referenced this pull request Jul 25, 2023
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.

2 participants