New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Never add user talk pages to watchlist #76

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@enterprisey
Member

enterprisey commented Feb 11, 2016

Putting this in a pull request so I can test it when I get to a computer with git.

@hasteur

This comment has been minimized.

Show comment
Hide comment
@hasteur

hasteur Feb 11, 2016

Member

I'm not wild about not doing this as I can see a conflicting scenario:

As a reviewer, if you're creating the user's talkpage with a review action, it would probably be a good idea to watch their talk page in case other warnings/notices come in.

Technically this looks correct. Is there a request for this?

Member

hasteur commented on 1e69163 Feb 11, 2016

I'm not wild about not doing this as I can see a conflicting scenario:

As a reviewer, if you're creating the user's talkpage with a review action, it would probably be a good idea to watch their talk page in case other warnings/notices come in.

Technically this looks correct. Is there a request for this?

This comment has been minimized.

Show comment
Hide comment
@enterprisey

enterprisey Feb 11, 2016

Member
Member

enterprisey replied Feb 11, 2016

This comment has been minimized.

Show comment
Hide comment
@hasteur

hasteur Feb 11, 2016

Member

Link to the discussion?

I still think a hard disabling this is a bad idea

Member

hasteur replied Feb 11, 2016

Link to the discussion?

I still think a hard disabling this is a bad idea

This comment has been minimized.

Show comment
Hide comment
@enterprisey

enterprisey Feb 11, 2016

Member

I suppose we could also make it a new preference, but I doubt the preference would see much use.

Member

enterprisey replied Feb 11, 2016

I suppose we could also make it a new preference, but I doubt the preference would see much use.

@hasteur

This comment has been minimized.

Show comment
Hide comment
@hasteur

hasteur Feb 11, 2016

Member

I'm not wild about not doing this as I can see a conflicting scenario:

As a reviewer, if you're creating the user's talkpage with a review action, it would probably be a good idea to watch their talk page in case other warnings/notices come in.

Technically this looks correct. Is there a request for this?

Member

hasteur commented on 1e69163 Feb 11, 2016

I'm not wild about not doing this as I can see a conflicting scenario:

As a reviewer, if you're creating the user's talkpage with a review action, it would probably be a good idea to watch their talk page in case other warnings/notices come in.

Technically this looks correct. Is there a request for this?

This comment has been minimized.

Show comment
Hide comment
@enterprisey

enterprisey Feb 11, 2016

Member
Member

enterprisey replied Feb 11, 2016

This comment has been minimized.

Show comment
Hide comment
@hasteur

hasteur Feb 11, 2016

Member

Link to the discussion?

I still think a hard disabling this is a bad idea

Member

hasteur replied Feb 11, 2016

Link to the discussion?

I still think a hard disabling this is a bad idea

This comment has been minimized.

Show comment
Hide comment
@enterprisey

enterprisey Feb 11, 2016

Member

I suppose we could also make it a new preference, but I doubt the preference would see much use.

Member

enterprisey replied Feb 11, 2016

I suppose we could also make it a new preference, but I doubt the preference would see much use.

@hasteur

This comment has been minimized.

Show comment
Hide comment
@hasteur

hasteur Feb 11, 2016

Member

I acutally expect this kind of behavior, and I'd like to see a larger consensus for this prior to it being enacted. I'd bet that other reviewers expect the previous behavior over the behavior of the others.

Member

hasteur commented Feb 11, 2016

I acutally expect this kind of behavior, and I'd like to see a larger consensus for this prior to it being enacted. I'd bet that other reviewers expect the previous behavior over the behavior of the others.

@enterprisey

This comment has been minimized.

Show comment
Hide comment
@enterprisey

enterprisey May 29, 2017

Member

Gonna abandon this patch in favor of more UI work.

Member

enterprisey commented May 29, 2017

Gonna abandon this patch in favor of more UI work.

@Technical-13

This comment has been minimized.

Show comment
Hide comment
@Technical-13

Technical-13 Jul 13, 2017

I'd oppose this as well... It's impossible to watch a user page and not watch the associate talk page. If the draft is in the user's userspace, which I'm assuming is still a valid option, it would require a core MW change to be able to watch the draft and not the talk. I also agree with @hasteur that reviewers should watch talk pages for responses to notices or other actions related to the review.

Technical-13 commented Jul 13, 2017

I'd oppose this as well... It's impossible to watch a user page and not watch the associate talk page. If the draft is in the user's userspace, which I'm assuming is still a valid option, it would require a core MW change to be able to watch the draft and not the talk. I also agree with @hasteur that reviewers should watch talk pages for responses to notices or other actions related to the review.

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