Zenodo: Text search (communities): case sensitive

Created on 9 Feb 2017 · 9Comments · Source: zenodo/zenodo

When searching for communities, if you don't type the community title exactly as it was defined by the creator, no chance find what you're looking for.
Check by yourself:

Search for "human brain" --> no result
Search for "Human Brain" --> expected result

Very, very restrictive in my opinion.

Source

par4dise

All 9 comments

I would like to work on this issue.

jainaman224 on 28 Feb 2017

@jainaman224 Feel free to take a look. I don't think anyone else is looking into this right now.

krzysztof on 28 Feb 2017

👍1

@jainaman224 This is related to https://github.com/inveniosoftware/invenio-communities/issues/99, which as you can see though resides in another repository of the Invenio framework that Zenodo is based on. This will involve pushing changes to invenio-communities. To start, first fork the invenio-communities repository, fix the issue locally, test it, and once you're done let us know here (include a URL to your fork). We will then take a look at it and help you open a PR on invenio-communities.

slint on 28 Feb 2017

@slint Thank you.

jainaman224 on 28 Feb 2017

@slint Please have a look at https://github.com/jainaman224/invenio-communities/tree/patch-1.

jainaman224 on 1 Mar 2017

👍1

@jainaman224 Looks good! It would be also great if you could also write some basic tests for these changes. Take some inspiration from the already existing tests in the tests directory, and create a similar test_models.py file, where you test some basic use cases for your changes.

slint on 1 Mar 2017

👍1

@slint @krzysztof Please have a look at https://github.com/inveniosoftware/invenio-communities/pull/100.

jainaman224 on 2 Mar 2017

The PR has been merged: https://github.com/inveniosoftware/invenio-communities/pull/100

invenio-communities release and integration is still needed.

krzysztof on 7 Mar 2017

Closing in favour of #1050