[FIX] Updated conditions for showing the `OpenNeuro` alert - Githubissues

neurobagel / old-query-tool

User interface for searching across Neurobagel graph

https://query.neurobagel.org/

MIT License

2 stars 1 forks source link

[FIX] Updated conditions for showing the `OpenNeuro` alert #334

Closed rmanaem closed 9 months ago

rmanaem commented 9 months ago

Closes #333

Changes proposed in this pull request:

Updated queryingOpenNeuro computed property to only show the OpenNeuro alert if
- It is in the available nodes and All is selected
- It is in the selected nodes
- alertDismiss is false

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:

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

netlify[bot] commented 9 months ago

Deploy Preview for neurobagel-query-tool ready!

Name	Link
Latest commit	11717519bb05a332ede35f29e0e190c0be1745bd
Latest deploy log	https://app.netlify.com/sites/neurobagel-query-tool/deploys/657fb3d968601b0008cfa921
Deploy Preview	https://deploy-preview-334--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.