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

Display all the images of a category together using FastCCI #2393

Open
ShridharGoel opened this issue Feb 2, 2019 · 6 comments
Open

Display all the images of a category together using FastCCI #2393

ShridharGoel opened this issue Feb 2, 2019 · 6 comments

Comments

@ShridharGoel
Copy link
Contributor

Summary:

Suppose an image is present in a particular subcategory which corresponds to a particular category. Currently, the app would show the image only in that subcategory and not in the list of images of the category.

If some image is in a subcategory, it should also be shown in the parent category of that subcategory. This can be implemented using FastCCI.

Steps to reproduce:

  1. Go to Explore.
  2. Search for something.
  3. Go to a category. Media tab would show the images.
  4. Click on the subcategories tab and then go to a subcategory.
  5. Not all images of the subcategory would have been present in the category.

Commons app version:

Latest master

Would you like to work on the issue?

Yes

@ShridharGoel
Copy link
Contributor Author

@misaochan @nicolas-raoul @maskaravivek @neslihanturan Your views on this?

@nicolas-raoul
Copy link
Member

I totally agree!
Using FastCCI you could even display the best (featured/etc) images first.

@misaochan
Copy link
Member

Tentatively sounds good to me. However, the use of FastCCI would require a complete overhaul of our current Commons query, right? Do we know how reliable the database is?

@nicolas-raoul
Copy link
Member

Last year it was not reliable, with months-long downtimes :-/

@misaochan
Copy link
Member

Hm, in that case is it a good idea to do this? We would need a very thorough fallback mechanism if we do.

@ShridharGoel
Copy link
Contributor Author

We'll need to set a mechanism where it would use the current system if the FastCCI servers go down, although I think that would rarely happen.

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

No branches or pull requests

3 participants