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
legacy virtual ontology id to acronym resolver broke #250
Comments
Is it even worth fixing this issue? virtual ontology ids are legacy and have not been in use for a number of years. |
I know the reason for the breakage, and it shouldn't be a difficult fix. I'll take a look at this tomorrow. |
I agree though, that having ontology ID resolver code in the Rails app feels a little messy. If there are better alternatives, I'm all ears. |
we could look at the logs and determine which virtual ids are still in use and add redirects on the web server level. After this, we could remove the resolver from the rails. There is no need for it in the ontoportal appliance anyway. |
@graybeal expressed a preference that we continue to accept the old numeric identifiers and resolve them to the corresponding acronyms in our current system. So, I fixed the code because it wasn't a significant effort. I agree that we should use a better solution for resolving the old IDs, but I think spending time on this should be a lower priority than a lot of the other things we have to work on. In the event we decide to tackle this, I think the work should be tracked under a new ticket that's characterized as an enhancement request. |
virtual (legacy numeric based ontology id) to acronym resolver no longer works. UI throws 500 error when browsing
bioportal.bioontology.org/ontologies/1032 which used to redirect to bioportal.bioontology.org/ontologies/NCIT
The text was updated successfully, but these errors were encountered: