Module talk:Protected edit request

This is an old revision of this page, as edited by Lam-ang (talk | contribs) at 16:55, 11 December 2020 (Undid revision 993610229 by 204.184.37.5 (talk)). The present address (URL) is a permanent link to this revision, which may differ significantly from the current revision.

Latest comment: 4 years ago by MSGJ in topic Feature request

Inactive requests

Answered requests ought to show the names of the templates just as active requests do.

I've just answered a request at Template talk:Infobox church#Template-protected edit request on 8 August 2018. Now that the request has been answered there's nothing to show that the request was actually for Template:Infobox church/denomination.

Similar problem exists for recent requests...

My best effort would be to clone Module:Protected edit request/active to Module:Protected edit request/inactive and start from there, but I'm sure there's got to be a better fix than that. Cabayi (talk) 14:51, 8 August 2018 (UTC)Reply

Just noting that this was   Fixed per #Template-protected edit request on 11 March 2020 below. SD0001 (talk) 03:34, 12 March 2020 (UTC)Reply

Template-protected edit request on 11 March 2020

Please sync changes from Module:Protected edit request/sandbox.

Improvement made: the template will show the names of pages to which edits were requested even after the request has been marked as answered. This will be skipped if there was just one page in the request which is same as the subject page.

At present, if an edit request is being made for a page which isn't the subject page, the name of the page get hidden when it's marked as answered, which is quite confusing for a reader.

See testcases at User talk:SD0001/sandbox2. SD0001 (talk) 21:44, 11 March 2020 (UTC)Reply

  Done * Pppery * it has begun... 22:50, 11 March 2020 (UTC)Reply
I've repaired the links to categories and files. — JJMC89(T·C) 22:52, 15 March 2020 (UTC)Reply

Template-protected edit request on 30 May 2020

Please change line 34

	if not mw.title.getCurrentTitle().isTalkPage and not self.demo then

to

	if not mw.title.getCurrentTitle().isTalkPage and not self.args.demo then

to fix parameter |demo=. See Template:Edit template-protected/testcases#Answered for demonstration of the issue. This change fixed the error output in {{Edit template-protected/sandbox}}. —⁠andrybak (talk) 05:02, 30 May 2020 (UTC)Reply

  Done Cabayi (talk) 05:17, 30 May 2020 (UTC)Reply

Feature request

When a request to edit a TemplateStyles stylesheet is made, could the sandbox links point to Template:XYZ/sandbox/styles.css because I believe this is the convention. I guess it is currently looking for a sandbox in Template:XYZ/styles.css/sandbox which is not correct. Thanks — Martin (MSGJ · talk) 12:31, 11 September 2020 (UTC)Reply

@MSGJ:   Done Jackmcbarn (talk) 23:54, 11 September 2020 (UTC)Reply
Fantastic! Thanks — Martin (MSGJ · talk) 16:19, 13 September 2020 (UTC)Reply