Skip to content

Conversation

@evantk91
Copy link
Contributor

@evantk91 evantk91 commented Apr 7, 2023

🔹 Jira Ticket(s) if any

✏️ Description

This pull request puts logic setting base url in a separate method and sets the base url based on the data region config value.

@evantk91 evantk91 changed the base branch from master to eudc-updates April 7, 2023 22:12
@codecov
Copy link

codecov bot commented Apr 10, 2023

Codecov Report

❗ No coverage uploaded for pull request base (eudc-updates@58e6352). Click here to learn what that means.
Patch has no changes to coverable lines.

❗ Current head 6bf0d59 differs from pull request most recent head 3232df6. Consider uploading reports for the commit 3232df6 to get more accurate results

Additional details and impacted files
@@               Coverage Diff               @@
##             eudc-updates     #549   +/-   ##
===============================================
  Coverage                ?   44.12%           
===============================================
  Files                   ?       68           
  Lines                   ?     3973           
  Branches                ?      461           
===============================================
  Hits                    ?     1753           
  Misses                  ?     2013           
  Partials                ?      207           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@evantk91 evantk91 marked this pull request as draft April 11, 2023 15:37
@evantk91 evantk91 marked this pull request as ready for review April 18, 2023 17:53
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.

Really nice!

IterableLogger.v(TAG, ">>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>\n");
String baseUrl = (iterableApiRequest.baseUrl != null && !iterableApiRequest.baseUrl.isEmpty()) ? iterableApiRequest.baseUrl :
ITERABLE_BASE_URL;
String baseUrl = getBaseUrl();
Copy link
Member

Choose a reason for hiding this comment

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

Really good simplification!

*/
class IterableRequestTask extends AsyncTask<IterableApiRequest, Void, IterableApiResponse> {
static final String TAG = "IterableRequest";
static final String ITERABLE_BASE_URL = "https://api.iterable.com/api/";
Copy link
Member

Choose a reason for hiding this comment

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

👌

@evantk91 evantk91 merged commit c917eff into eudc-updates Apr 18, 2023
@evantk91 evantk91 deleted the evan/MOB-5934-set-base-endpoint-based-on-config branch April 18, 2023 18:15
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.

3 participants