Siddharth VP

Results 77 comments of Siddharth VP

It's certainly a bug that the PROD rationale is unrecoverable. We have the Morebits.status.printUserText system used in other modules like XFD to recover the text in case of error. As...

There's actually a more efficient way to use the API here. We can replace the current query for listing redirects with generator=redirects&prop=info&inprop=talkid – which returns essentially the same info but...

Very nice attempt, but can I suggest a different approach? [Module:Redirect template](https://en.wikipedia.org/wiki/Module:Redirect_template) supports a |class= parameter (which I implemented about two years ago with this very usecase in mind, at...

Well now that I go back and look at it, I see that |class= of Module:Redirect_template doesn't work the same way as |name= of Module:Message_box (which we use for untagging...

> What we'd need to do is to edit every rcat template to add |id={{subst:PAGENAME}} Done! All ~285 rcat templates using AWB. It was a lot easier than I thought...

I implemented the feature on [twinkle-core ](https://github.com/wikimedia-gadgets/twinkle-core/blob/master/src/tag.ts) / [twinkle-enwiki](https://github.com/wikimedia-gadgets/twinkle-enwiki/blob/master/src/tag.ts), centralising the tagging code for both article & redirect modes (can also be easily extended to file tags). Not sure if...

The code for untagging is deeply convoluted. Even more so for the article untagging part which is really really bad (I wrote that 2 years - certainly the last thing...

It's because the `ext.gadget.select2` RL module somehow hasn't loaded. Is this while using the `npm start` script? The script only does lazy load via mw.loader.load: https://github.com/wikimedia-gadgets/twinkle/blob/f05d99c689d8d0e8e96dfcfbe83c2bd64257645b/scripts/dev-server.js#L20 Wrapping `jsCode` with mw.loader.using...

Not quite. mw.loader.using returns a promise; we have to put all remaining code in a `.then()` callback, which ensures select2 will be available when the remaining code runs.