CSD logging #136

Closed
Technical-13 opened this Issue Aug 29, 2013 · 8 comments

Comments

Projects
None yet
3 participants
@Technical-13
Contributor

Technical-13 commented Aug 29, 2013

Now that we have the ability to tag files for deletion and log that to the user's Twinkle CSD log, logging should be added to all things we tag for CSD as well.

@theopolisme

This comment has been minimized.

Show comment
Hide comment
@theopolisme

theopolisme Aug 29, 2013

Contributor

"Tag files for deletion"? Assume you mean G13s... yeah, I suppose logging could be implemented.

Contributor

theopolisme commented Aug 29, 2013

"Tag files for deletion"? Assume you mean G13s... yeah, I suppose logging could be implemented.

@Technical-13

This comment has been minimized.

Show comment
Hide comment
@Technical-13

Technical-13 Aug 29, 2013

Contributor

No @theopolisme, I mean all things tagged for any CSD criterion (G12 & G13 only atm, but could be extended) should be logged to the reviewers CSD log.

Contributor

Technical-13 commented Aug 29, 2013

No @theopolisme, I mean all things tagged for any CSD criterion (G12 & G13 only atm, but could be extended) should be logged to the reviewers CSD log.

@theopolisme

This comment has been minimized.

Show comment
Hide comment
@theopolisme

theopolisme Aug 29, 2013

Contributor

"Now that we have the ability to tag files for deletion and log that to the user's Twinkle CSD log" should be changed to "Now that we have the ability to tag G13-eligible pages for deletion and log that to the user's Twinkle CSD log"...since right now only G13s are logged...eh, rather a moot point. Yes, I understood you.

Contributor

theopolisme commented Aug 29, 2013

"Now that we have the ability to tag files for deletion and log that to the user's Twinkle CSD log" should be changed to "Now that we have the ability to tag G13-eligible pages for deletion and log that to the user's Twinkle CSD log"...since right now only G13s are logged...eh, rather a moot point. Yes, I understood you.

@Technical-13

This comment has been minimized.

Show comment
Hide comment
@Technical-13

Technical-13 Aug 30, 2013

Contributor

All pages are just a file on a more root level... Yes.. moot point...

Contributor

Technical-13 commented Aug 30, 2013

All pages are just a file on a more root level... Yes.. moot point...

theopolisme added a commit that referenced this issue Aug 30, 2013

@theopolisme

This comment has been minimized.

Show comment
Hide comment
@theopolisme

theopolisme Aug 30, 2013

Contributor

@Technical-13 please test (in theobot) when you have a chance -- and by all means let me know if i did anything stupid!

Contributor

theopolisme commented Aug 30, 2013

@Technical-13 please test (in theobot) when you have a chance -- and by all means let me know if i did anything stupid!

@wikipedia-mabdul

This comment has been minimized.

Show comment
Hide comment
@wikipedia-mabdul

wikipedia-mabdul Sep 2, 2013

Member

just for interest (didn't checked any code nor tested) what happens if a user doesn't have yet created the log page (so just "installed" TW)? Does our code work well?

Member

wikipedia-mabdul commented Sep 2, 2013

just for interest (didn't checked any code nor tested) what happens if a user doesn't have yet created the log page (so just "installed" TW)? Does our code work well?

@Technical-13

This comment has been minimized.

Show comment
Hide comment
@Technical-13

Technical-13 Sep 2, 2013

Contributor

@wikipedia-mabdul I think it does everything except add the "This is a blah log. If you wish to stop logging and have it deleted, you may do so under [[CSD:U1]] blah blah". Which reminds me, we need a way for people to turn of logging if they so wish (not sure why anyone would, but whatever.).

Contributor

Technical-13 commented Sep 2, 2013

@wikipedia-mabdul I think it does everything except add the "This is a blah log. If you wish to stop logging and have it deleted, you may do so under [[CSD:U1]] blah blah". Which reminds me, we need a way for people to turn of logging if they so wish (not sure why anyone would, but whatever.).

@theopolisme

This comment has been minimized.

Show comment
Hide comment
@theopolisme

theopolisme Sep 2, 2013

Contributor

If the user does not have a log page already set up, then we do not log anything. Seems a bit too forward to actually set up a log page and everything...I'd rather the user initiate the logging on their own rather than us just shoving it at them.

Contributor

theopolisme commented Sep 2, 2013

If the user does not have a log page already set up, then we do not log anything. Seems a bit too forward to actually set up a log page and everything...I'd rather the user initiate the logging on their own rather than us just shoving it at them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment