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

Added links to random submissions #33

Merged
merged 2 commits into from Dec 10, 2014

Conversation

Projects
None yet
2 participants
@enterprisey
Member

enterprisey commented Dec 8, 2014

In lieu of actually fetching the pages, we can just use Special:RandomInCategory.

I've included 3 categories here (general, very old, and GFOO); reading the WT:WPAFC page, it seems like those three are the most commonly used.

Fixes #27.

@theopolisme

This comment has been minimized.

Show comment
Hide comment
@theopolisme

theopolisme Dec 10, 2014

Contributor

THIS IS AWESOME! Great thinking! :D

Can we remove the getNextSubmission() function then, perhaps?

Contributor

theopolisme commented Dec 10, 2014

THIS IS AWESOME! Great thinking! :D

Can we remove the getNextSubmission() function then, perhaps?

removed pointless code
i.e. code that was rendered pointless by the new way of displaying "next submission" links
@enterprisey

This comment has been minimized.

Show comment
Hide comment
@enterprisey

enterprisey Dec 10, 2014

Member

Removed that function and its commented-out call in the done-loading hook; merging.

Member

enterprisey commented Dec 10, 2014

Removed that function and its commented-out call in the done-loading hook; merging.

enterprisey added a commit that referenced this pull request Dec 10, 2014

@enterprisey enterprisey merged commit 948b203 into master Dec 10, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@enterprisey enterprisey deleted the random-submission-link branch Dec 10, 2014

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