Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Commit

Permalink
fix: added todo for youtube API key
Browse files Browse the repository at this point in the history
  • Loading branch information
omerhabib26 committed Jul 21, 2023
1 parent f262567 commit 60aaded
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ public static List<String> getRequestedFieldsList(@NonNull Config config) {
requestedFields = Collections.singletonList(
DiscussionRequestFields.PROFILE_IMAGE.getQueryParamValue());
} else {
requestedFields = Collections.EMPTY_LIST;
requestedFields = Collections.emptyList();

Check warning on line 43 in OpenEdXMobile/src/main/java/org/edx/mobile/model/discussion/DiscussionRequestFields.java

View check run for this annotation

Codecov / codecov/patch

OpenEdXMobile/src/main/java/org/edx/mobile/model/discussion/DiscussionRequestFields.java#L43

Added line #L43 was not covered by tests
}
return requestedFields;
}
Expand Down
3 changes: 3 additions & 0 deletions OpenEdXMobile/src/main/java/org/edx/mobile/util/Config.java
Original file line number Diff line number Diff line change
Expand Up @@ -452,6 +452,9 @@ public boolean isYoutubePlayerEnabled() {
return enabled;
}

// TODO we have removed the API key from Config as a part of
// https://2u-internal.atlassian.net/browse/LEARNER-8794, This code will be removed once
// https://2u-internal.atlassian.net/browse/LEARNER-9354 is completed and merged in master
public String getApiKey() {
return apiKey;
}
Expand Down

0 comments on commit 60aaded

Please sign in to comment.