Skip to content

Commit

Permalink
Merge pull request #529 from edx/ammar/send-enterprise-uuid-as-query-…
Browse files Browse the repository at this point in the history
…param-for-tableau

feat: send enterprise uuid as query param for tableau
  • Loading branch information
muhammad-ammar authored Apr 27, 2021
2 parents 82119a6 + dc38001 commit 1597b0d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/components/analytics/AnalyticsCharts.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ export default function AnalyticsCharts(enterpriseId) {
// Fetch token
useEffect(() => {
setIsLoading(true);
AnalyticsApiService.fetchTableauToken()
AnalyticsApiService.fetchTableauToken(enterpriseId)
.then((response) => {
setIsLoading(false);
setToken(response.data);
Expand Down
2 changes: 1 addition & 1 deletion src/components/analytics/data/service.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ class AnalyticsApiService {
const queryParams = {
...options,
};
return getAuthenticatedHttpClient().get(AnalyticsApiService.tableauTokenUrl);
return getAuthenticatedHttpClient().get(`${AnalyticsApiService.tableauTokenUrl}/${queryParams.enterpriseId}`);
}
}

Expand Down

0 comments on commit 1597b0d

Please sign in to comment.