Skip to content
This repository has been archived by the owner on Jan 24, 2019. It is now read-only.

Quick Discover Fix #1016

Open
xmatthewx opened this issue Apr 14, 2016 · 5 comments
Open

Quick Discover Fix #1016

xmatthewx opened this issue Apr 14, 2016 · 5 comments

Comments

@xmatthewx
Copy link

The Discover gallery does not show all featured projects. Several of us tried to track down the root of this "bug" (#1010), but it is not a bug after all. 😕

Cause: Part of the initial strategy was to promote local content, which is great. So if a user in a supported locale views Discover, they only see projects from other users in that locale. I assumed this was a goal, not a launch feature. My mistake.

Problem: A fledgling app without any content needs to fill its empty shelves before all else. Users in es-MX (a supported locale), still see zero projects in Discover. Users in other supported locales will not see any of the helper how-to projects I created. Users in unsupported locales actually have a better view than the rest of us.

Unfortunately, this likely had some impact on post-launch performance. Fortunately, we're ready to launch the work completed in January. I believe @gvn is interested in building the release (and sharing the process with others who are interested in learning).

As a quick fix, I recommend we strip lang from our Discover route. I believe it sits here: https://github.com/mozilla/webmaker-core/blob/develop/src/components/project-list/project-list.jsx#L79. What do you think? A more thorough fix would enable us to feature a project in one or all locales, but this would require more code and more moderation than we can probably muster. Add your thoughts below.

It will be interesting to see the impact of this quick fix coupled with the introduction hashtags.

@vazquez @alanmoo @cadecairos @ryanwse

@xmatthewx xmatthewx added this to the 1.6.2 milestone Apr 14, 2016
This was referenced Apr 14, 2016
@ryanwarsaw
Copy link
Contributor

What are your thoughts on providing a user toggle-able option that allows them to choose whether they want to only see localized content or not? Surely we would need to build such a view in the future anyways (for other user related options).

@xmatthewx
Copy link
Author

@ryanwse – you're right, that could be an ideal solution. We just don't have the time to design, build, test, and localize it. Adding a toggle is relatively easy. Figuring out where to put it, how to label it, and what to call it gets more complicated. Would sketch up a proposal to consider for later. In the meantime, Hashtags will also give people a chance to create a connection between local content.

@xmatthewx xmatthewx modified the milestones: 1.7.0, 1.6.1 May 3, 2016
@xmatthewx
Copy link
Author

As a quick demo of the problem, hit both of these requests. You'll see that the english route doesn't return project ID 56558.

https://api.webmaker.org/discover/fr-US?page=1&count=50
https://api.webmaker.org/discover/en-US?page=1&count=50
https://api.webmaker.org/discover/?page=1&count=50

see: line 79 and line 13 in the project-list component.

@cadecairos
Copy link
Contributor

cadecairos commented May 3, 2016

Project 56558 shouldn't show up in the en-US route - it's lang is set to fr

edit - so is the problem is that you can't see a non-localized set of discover projects without changing the system locale?

@xmatthewx
Copy link
Author

You are right that it's not a bug in the code. We don't have enough curators to feature activity, so discover is likely to feel stagnant. And, helper and how-to projects haven't been localized, so others don't see them.

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

No branches or pull requests

3 participants