[FIX] Refactored `submitQuery` to resolve the query url construction bug - Githubissues

neurobagel / old-query-tool

User interface for searching across Neurobagel graph

https://query.neurobagel.org/

MIT License

2 stars 0 forks source link

[FIX] Refactored `submitQuery` to resolve the query url construction bug #317

Closed rmanaem closed 11 months ago

rmanaem commented 11 months ago

Closes #316

Changes proposed in this pull request:

Refactored submitQuery into two methods
- constructQueryURL method which will use URLSearchParams to construct the query param portion of the query URL
- submitQuery will simply call constructQueryURL and then make the axios call
Updated APIRequest test

Checklist

[x] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) _(see https://neurobagel.org/contributing/pull_requests for more info)_
[x] PR links to GitHub issue with mention Closes #XXXX
[x] Tests pass
[x] Checks pass

For new features:

[ ] Tests have been added

For bug fixes:

[x] There is at least one test that would fail under the original bug conditions.

netlify[bot] commented 11 months ago

Deploy Preview for neurobagel-query-tool ready!

Name	Link
Latest commit	8bda1e90744caebf90725e332ebad189fe7f8ffb
Latest deploy log	https://app.netlify.com/sites/neurobagel-query-tool/deploys/6571e709e516460008bcd50a
Deploy Preview	https://deploy-preview-317--neurobagel-query-tool.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.