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.

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

Feature/terminology #56

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

dconlan
Copy link
Collaborator

@dconlan dconlan commented Jul 14, 2022

Hi Orion,

I've created this feature based on your old selectize branch. It adds a number of ways to use alternative terminology providers to do autocomplete on disorders, phenotypes and genes. I've written a bit of documentation on it in the terminology.md file.

I've not tested the CTSS terminology as I haven't ever setup a phenotips server for it to use.

I'm not sure the best way to deal with things like the bioportal api key or a fhir terminology server that needs authentication.
I do have some code thats part of my redcap plugin that uses redcap as a proxy and it hides the keys and does the authentication. But I didn't include it in this pull request as its tightly coupled to the redcap plugin and so is not relevant to others.

David Conlan

buske and others added 3 commits November 12, 2019 01:28
There is an issue with only the first few terms being recorded, and not being
able to be deleted. It appears that the NodeMenu _updating flag is being
erroneously set, or for some other reason the subsequent changes are being
ignored.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants