AdvancedFlagging icon indicating copy to clipboard operation
AdvancedFlagging copied to clipboard

Added support for comments on other sites than SO, added placeholders, fixed some bugs

Open fastnlight0 opened this issue 1 year ago • 2 comments

Added support for comments and Answer related NAA options for sites other than Stack Overflow.

In addition, similar to AutoReviewComments, I added 3 placeholders:

Placeholder Description Example
%SITENAME% The site name Stack Overflow, Super User, Physics Stack Exchange
%SITEURL% The domain of the current site stackoverflow.com, money.stackexchange.com
%OP% The author of the question/answer fastnlight

I've also fixed a bug where if a user was recognized in a collective, AdvancedFlagging would believe their answer was actually a question.

fastnlight0 avatar Jan 17 '24 02:01 fastnlight0

Hmm... I'm reluctant to approve this. IIRC, the reason that adding answer-related comments via the AF menu on non-SO sites was disabled is because of this Charcoal policy:

You should not leave comments under a post reported by SmokeDetector, unless you are otherwise familiar with that Stack Exchange site. In general, that means you participate on the site for more than responding to SmokeDetector reports and/or viewing a few Hot Network Questions.

double-beep avatar Jan 25 '24 13:01 double-beep

The primary reason I allowed comments on all sites is so AF isn't a SO only userscript, but could be used by regulars of other sites. I fully understand why it was disabled in the first place. If you want me to disable it again, please let me know.

fastnlight0 avatar Jan 25 '24 13:01 fastnlight0

I'm sorry for the delay in responding. Since I've rewritten some parts of the code this PR modifies, could you open a new PR with your changes? Thanks!

double-beep avatar Jun 03 '24 10:06 double-beep

@double-beep Done! #292

fastnlight0 avatar Jun 03 '24 14:06 fastnlight0