Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove query tool specific configs from API-graph environment variable table #226

Closed
2 of 3 tasks
alyssadai opened this issue Nov 22, 2023 · 0 comments · Fixed by #238
Closed
2 of 3 tasks

Remove query tool specific configs from API-graph environment variable table #226

alyssadai opened this issue Nov 22, 2023 · 0 comments · Fixed by #238
Assignees
@surchs
Labels
documentation Changes only affect the documentation quick fix Minimal planning and/or implementation work required.

Comments

@alyssadai
Copy link
Contributor

alyssadai commented Nov 22, 2023 •
edited by surchs
Loading

See also neurobagel/documentation#133

TODO

  • all config files about non-API stuff are gone from repo
  • separate link to docs from README on "deploy query tool with n-API"
  • in the docs, the env-var table is updated to no longer show variables that were needed when n-API + query tool were deployed together
@alyssadai alyssadai added the documentation Changes only affect the documentation label Nov 22, 2023
@alyssadai alyssadai added the flag:schedule Flag issue that should go on the roadmap or backlog. label Nov 22, 2023
@rmanaem rmanaem moved this to Backlog in Neurobagel Nov 22, 2023
@rmanaem rmanaem added quick fix Minimal planning and/or implementation work required. and removed flag:schedule Flag issue that should go on the roadmap or backlog. labels Nov 22, 2023
@surchs surchs moved this from Backlog to Specify - Active in Neurobagel Dec 6, 2023
@surchs surchs moved this from Specify - Active to Specify - Done in Neurobagel Dec 6, 2023
@surchs surchs moved this from Specify - Done to Implement - Active in Neurobagel Dec 6, 2023
@surchs surchs self-assigned this Dec 6, 2023
surchs added a commit that referenced this issue Dec 6, 2023
@surchs
[FIX] Remove query tool config
2c75bbb 
Addresses #226
@surchs surchs moved this from Implement - Active to Implement - Done in Neurobagel Dec 6, 2023
@surchs surchs mentioned this issue Dec 6, 2023
Merged
6 tasks
@rmanaem rmanaem moved this from Implement - Done to Review - Active in Neurobagel Dec 6, 2023
surchs added a commit that referenced this issue Dec 6, 2023
@surchs
[FIX] Remove query tool config (#238)
df4353a 
Addresses #226
@github-project-automation github-project-automation bot moved this from Review - Active to Review - Done in Neurobagel Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@surchs surchs

Labels
documentation Changes only affect the documentation quick fix Minimal planning and/or implementation work required.
Projects
Neurobagel
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[FIX] Remove query tool config neurobagel/api
3 participants
@surchs @alyssadai @rmanaem