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

search by SHA or DOI #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

stuartlynn
Copy link

Adding a search by DOI if the search by SHA fails

@arfon
Copy link
Member

arfon commented Dec 10, 2014

✨ a test would be good here.

@stuartlynn
Copy link
Author

Humm this is more complicated than I thought. The dois look like 10.5072/zenodo.46 which we could capture with a route like /ideas/:prefix/:suffix but the .46 on the end is going to confuse and trigger the rails format selector I think. Not sure how to overcome that in a non hacky way.

@lnielsen
Copy link
Contributor

It's not possible just to grab everything below e.g. /ideas/doi/ with rails?

@arfon
Copy link
Member

arfon commented Dec 17, 2014

I think it must be. I think we need to override the content matchers for some urls.

On Wed, Dec 17, 2014 at 6:10 AM, Lars Holm Nielsen
[email protected] wrote:

It's not possible just to grab everything below e.g. /ideas/doi/ with rails?

Reply to this email directly or view it on GitHub:
#34 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants