search

neurobagel / documentation

The neurobagel documentation
http://neurobagel.org/
MIT License
10 stars 6 forks source link

[MNT] Deprecate non-federated query tool config, bump Docker Compose required ver. #217

Closed alyssadai closed 2 months ago

alyssadai commented 2 months ago

Changes proposed in this pull request:

Checklist

Please leave checkboxes empty for PR reviewers

netlify[bot] commented 2 months ago

Deploy Preview for neurobagel-documentation ready!

Name Link
Latest commit d72b1f9509e9c2fc0d0096b382001d893a17e17b
Latest deploy log https://app.netlify.com/sites/neurobagel-documentation/deploys/66b650905bbbcd00081ea3f8
Deploy Preview https://deploy-preview-217--neurobagel-documentation.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

alyssadai commented 2 months ago

Hey @surchs, just to wrap up a small stale issue into this PR, I added a commit https://github.com/neurobagel/documentation/pull/217/commits/c92a4ab3819fdd00825c63c84cce06d694253ad2 to address Barbara's request for more clarity concerning local_nb_nodes.json https://github.com/neurobagel/documentation/issues/204 (since I'm editing the config page anyways).

Let me know if you think the addition is clear enough that local_nb_nodes.json should only include n-APIs.

surchs commented 2 months ago

Hey @surchs, just to wrap up a small stale issue into this PR, I added a commit c92a4ab to address Barbara's request for more clarity concerning local_nb_nodes.json #204 (since I'm editing the config page anyways).

Let me know if you think the addition is clear enough that local_nb_nodes.json should only include n-APIs.

Yep, that is a good way to address it I think. Thanks for that!