Skip to content

Commit

Permalink
Merge pull request #9 from ContinuumIO/CORE-5238/bugfix-metabase-get-…
Browse files Browse the repository at this point in the history
…databases-params

[CORE-5238] Use string for database endpoint params
  • Loading branch information
AlbertDeFusco authored Jan 30, 2024
2 parents 05b3234 + 5927d76 commit 09ad56c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion intake_metabase/source.py
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ def get_databases(self):
self._create_or_refresh_token()

headers = _merge_dicts({'X-Metabase-Session': self._token}, self.extra_headers)
params = json.dumps({'include': 'tables', 'saved': True})
params = {'include': 'tables', 'saved': 'true'}

res = requests.get(
urljoin(self.domain, '/api/database'),
Expand Down

0 comments on commit 09ad56c

Please # to comment.