Skip to content

Don't use locked state as default privacy setting in anywhere#4082

Closed
unarist wants to merge 1 commit intomastodon:masterfrom
unarist:fix-locked
Closed

Don't use locked state as default privacy setting in anywhere#4082
unarist wants to merge 1 commit intomastodon:masterfrom
unarist:fix-locked

Conversation

@unarist
Copy link
Copy Markdown
Contributor

@unarist unarist commented Jul 5, 2017

Currently we set default post privacy for locked account to "private". However, it only means:

  • default value for visibility option on post status API
  • default value for default privacy setting for web UI

i.e. it makes no sense if the client specifies (own default) visibility option, or the user already set default privacy setting. This may confuse users.

This removes the hint text about this from Profile page, and current usages for consistency.

Other option: Set default privacy setting for web UI to "private" when the user locked account, and provide this value to clients, although clients may not respect that value.

Currently we set *default post privacy* for locked account to "private". However, it only means:

* default value for `visibility` option on post status API
* default value for default privacy setting on web UI

i.e. it makes no sense if the client specifies (own default) visibility option, or the user already set default privacy setting. This may confuse users.

This removes the hint text about this from Profile page, and current usages for consistency.
@unarist
Copy link
Copy Markdown
Contributor Author

unarist commented Jul 5, 2017

Oops, this must be fixed since this actually changes default privacy setting of locked accounts. It needs some migration if we will go with current implementation.

@nightpool
Copy link
Copy Markdown
Member

This feels like throwing out the baby with the bathwater... why not just merge #4075? That feels like a much much simpler solution

@unarist
Copy link
Copy Markdown
Contributor Author

unarist commented Jul 6, 2017

@nightpool Providing default privacy settings via API is also good, but main concern is below hint text.

Requires you to manually approve followers and defaults post privacy to followers-only

I thought "defaults post privacy to followers-only" means changing default post privacy for web UI, but it's not correct. So I'm considering to take one of following options:

  • Remove that sentence. And this makes existing behaviors to undocumented, so I've removed those behaviors.
  • Change default post privacy for web UI on account locking, even if the user already set default post privacy.

This PR took first option, but now I think second option might be better.

@clworld clworld mentioned this pull request Jul 10, 2017
1 task
@Gargron
Copy link
Copy Markdown
Member

Gargron commented Jul 11, 2017

Should we rethink this now that #4075 is merged?

@unarist
Copy link
Copy Markdown
Contributor Author

unarist commented Jul 11, 2017

@Gargron Yeah. So, should account locking changes current settings only if they haven't configured (current behavior), or anytime? Users may not want to override their own settings, but override anytime seems understandable and easy to describe I think.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants