Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

add a tpa clean command support. #48

Closed
honnza opened this issue Dec 23, 2014 · 8 comments
Closed

add a tpa clean command support. #48

honnza opened this issue Dec 23, 2014 · 8 comments

Comments

@honnza
Copy link
Collaborator

honnza commented Dec 23, 2014

Currently, the only way to announce an offensive post is the tpa command, possibly coupled with a why command. This, however, doesn't clean the message if it was detected as LQ instead of as offensive. Moreover, tpa after a `clean command reverses the effects of cleaning (bug report incoming).

Please add a way to clean and announce a message in one go. Bonus points if it counts as an offensive TP report.

@ghost
Copy link

ghost commented Dec 23, 2014

This may require specific syntax. e.g., would both tpa clean why and tpa why clean be accepted?

Also: #39 is related.

@honnza
Copy link
Collaborator Author

honnza commented Dec 23, 2014

Why not support both? Ideally, why not support arbitrary chaining of commands?

@ghost
Copy link

ghost commented Dec 23, 2014

Another FR, I suppose. But contextual flags require a specific order. (Such as in #39)

@honnza
Copy link
Collaborator Author

honnza commented Dec 23, 2014

maybe require the reason modifier to be either last or have its argument quoted? I'm also OK with semicolons between modifiers.

@ghost
Copy link

ghost commented Dec 23, 2014

Perhaps this could be the order: tpa "offensive" clean why

@ArcticEcho
Copy link
Owner

So what's wrong with the currently existing tpa clean command?

@ghost
Copy link

ghost commented Dec 28, 2014

Example: called and reflected.

The title was not successfully cleaned.

@ArcticEcho
Copy link
Owner

There, fixed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants