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

Update the random title endpoint to use wikifeeds #1238

Merged
merged 2 commits into from
Feb 7, 2020

Conversation

mdholloway
Copy link
Contributor

The wikifeeds service has been updated to serve the /page/random/title
endpoint, which is consumed by the apps for the Explore feed. This
updates RESTBase to obtain it from wikifeeds. After this change is live,
the random page code will be removed from mobileapps.

This test was broken by a change to mobileapps to more aggressively
strip empty sections in mobile-html. Some section body text is added to
the input to fix it.
@Pchelolo
Copy link
Contributor

Pchelolo commented Feb 6, 2020

LGTM. Please rebase on #1240 to make the travis pass

The wikifeeds service has been updated to serve the /page/random/title
endpoint, which is consumed by the apps for the Explore feed. This
updates RESTBase to obtain it from wikifeeds. After this change is live,
the random page code will be removed from mobileapps.
@Pchelolo Pchelolo merged commit 584812c into wikimedia:master Feb 7, 2020
@mdholloway mdholloway deleted the random-wikifeeds branch June 30, 2020 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants