User talk:Didicodes/Sandbox/API:Random

This page used the Structured Discussions extension to give structured discussions. It has since been converted to wikitext, so the content and history here are only an approximation of what was actually displayed at the time these comments were made.

Feedback 1.0

Latest comment: 7 years ago2 comments2 people in discussion

Sweet! Here is first round of feedback:

Parameter history is missing information about parameter that was deprecated in 1.26
Embed the automated documentation and not link to it
Let the first sample code be about how to get 2 random pages and the second one be about getting random set of pages including information such as titles, content, images, etc. Found this interesting link on Stackoverflow here: https://stackoverflow.com/questions/33614492/wikipedia-api-get-random-pages which talks about this use case and for which the final query would look like this: https://en.wikipedia.org/w/api.php?format=json&action=query&generator=random&grnnamespace=0&prop=revisions%7Cimages&rvprop=content&grnlimit=10.
You could actually get rid of the additional notes section completely, as some of the information in there is in the automated docs and with the query above we will explain well how this module can be used as a generator..

May be you could make changes to the pull request accordingly and then I will take a look at it? User:SSethi_(WMF) 01:05, 23 October 2018 (UTC)Reply

Cool, Thanks for the feedback.

point 2: I actually tried embedding the automated document itself but the module could not be found, I checked other APi's and I realized it is just like APi:GeoSearch which also produces this same error when you embedded it directly, thats why I made the documentation a link. I will send you a screenshot on zulip.

point 1: rnredirect was introduced in v1.14 and Deprecated in v1.26 so is this format okay?

v1.14: Introduced rnredirect
v1.26: Deprecated rnredirect
Didicodes (talk) 04:36, 23 October 2018 (UTC)Reply