Topic on User talk:Zaycodes/Sandbox/API:Mergehistory

Thank you for working on this. Here's some feedback:

• On the first line, do not capitalize the R in request.
• Since we only have one sample code for this module, remove the example whose sample code is not merged.
• Create a sub page of this page to host the sample code and then transclude that page into this page. Name the page Sample code 1. Look at API:Tags and API:Tags/Sample_code_1 for reference.
• I think the sentence you have under "Demo apps" would be more appropriate if it were under "Additional notes" since the page Special:MergeHistory is not a demo app.

Hi @Jeropbrenda i've worked on all the feedbacks, kindly review

@Zaycodes Thank you for making the changes, however, the Sample code section is still not using a tabbed window as it should. See API:Documentation template#Sample code

Looking at https://www.mediawiki.org/wiki/API:Mergehistory, I can see that it had been translated to many languages. So that we don't lose the translations, make sure to add translation tags to your sandbox page.

Hi @Jeropbrenda, I've tried all the possible means i could think of but the tabbed window for the sample codes doesn't seem to work on my Sandbox page, i even copied exactly the same format from the template but it didn't work, i also used the tabbed window template, still didn't work. Don't have any idea what to do on that anymore.

Secondly I'm not sure what you mean when you say I should add translation tags to my sandbox page

@Zaycodes I'll look into the tabbed window issue. Don't worry about it. For the translation tags, I mean that since the contents of this page are going to be pasted into the original page, we need to retain the translations that the original page had. For that, we need to retain translation tags where there were the previous page had them. Please see one of the project resources: https://www.mediawiki.org/wiki/Help:Extension:Translate/Page_translation_example

Hi @Jeropbrenda, I've added translation tags, kindly review

@Zaycodes The changes are looking good, although I'm wondering if we need

__FORCETOC__

at the bottom of the page. Also, under the "See also" section, add a short description of the link you've listed. When you're done with this, you may paste the contents of this page to the main API:Mergehistory page. In the edit summary, make sure to mention something like "Updated as per phab:T232816 API documentation improvements". Ping me when you're done :)

Hi @Jeropbrenda, I've pasted the contents into the main page and also created a page for the sample codes

@Zaycodes Looks good! You may move your name to the "Done" column

Yay, thanks @Jeropbrenda. Can we keep the sandbox page until i finish working on stashedit or what do i do so i don't loose my work on deleting this page.

@Zaycodes Sure, no problem :)