When accepting a page over a redirect when the reviewer is an admin, the script should delete the target instead of tagging it for deletion

WPAFC / afch

Yet another Articles for creation helper script -- ACTIVE DEVELOPMENT NO LONGER HAPPENS IN THIS REPOSITORY AND HAS MOVED TO

https://github.com/WPAFC/afch-rewrite

9 stars 3 forks source link

When accepting a page over a redirect when the reviewer is an admin, the script should delete the target instead of tagging it for deletion #147

Closed earwig closed 11 years ago

earwig commented 11 years ago

See https://en.wikipedia.org/w/index.php?diff=571412960&oldid=571323405, which the script did automatically when I tried to accept the page. This feels unnecessary since I could've just deleted it myself (which I did).

theopolisme commented 11 years ago

Yeah, that's in a !todo comment in the source https://github.com/WPAFC/afch/blob/abb1816d93d992e8e98ea2bfde56b33ae1cac634/src/submissions.js#L1037. Will work on this...

theopolisme commented 11 years ago

I've started working on afcHelper_deletePage() to handle this.

Technical-13 commented 11 years ago

@theopolisme chill, it was not intentional to mess with you. AGF man... See other post where you said I was mistaken. There was a lack of communication and I shifted some labels around, had second thoughts and unshifted them awaiting to talk to you about it.

theopolisme commented 11 years ago

I realized that and deleted my comment before you replied, actually. Sorry.

Technical-13 commented 11 years ago

No worries, justt remember, I don't mute any threads, so I get emails for everything.