Skip to content
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

アカウント削除機能、コミュニティからの退会の扱いについて #108

Open
inafact opened this issue May 13, 2019 · 7 comments
Assignees
Labels
enhancement New feature or request question Further information is requested
Milestone

Comments

@inafact
Copy link
Member

inafact commented May 13, 2019

@hiroyuki @thirozumi

以前多少話していて、デザイン上では明確にUIが用意されておらずふわっとしてしまっていたので、こちらの機能についてどういう扱いにするべきか相談です。

現状、いずれもAPI的には実装済・実行可能で、UIとしては

  • コミュニティの退会 → コミュニティトップ右上切り替え・追加ボタンで開くモダールビューで選択済みのコミュニティを再選択する
  • アカウント削除→プロフィール/その他/アカウントの削除(開発者・テスター向け)をタップで確認ダイアログが開くので了承する

となっています。
これらの機能を使えるようにしておく場合、弊害として一部リストビューの表示で削除・退会したユーザーのユーザー名が参照できなくなり、タイトルバーや一部コンテンツのテキスト要素が空になる場合があります。

@inafact inafact added enhancement New feature or request question Further information is requested labels May 13, 2019
@inafact
Copy link
Member Author

inafact commented May 13, 2019

Simulator Screen Shot - iPhone XS Max - 2019-05-13 at 18 34 29

Simulator Screen Shot - iPhone XS Max - 2019-05-13 at 18 34 49

Simulator Screen Shot - iPhone XS Max - 2019-05-13 at 18 35 02

@hiroyuki
Copy link

@inafact @thirozumi

いずれの機能も残しておきたいですが、消された奴は非表示にするか、アカウントが存在しませんぐらいにできないでしょうか?

@inafact
Copy link
Member Author

inafact commented May 13, 2019

@hiroyuki そうですね、その辺の代替テキストを入れとくのが妥当かなとは思います。

非表示のほうは、ユーザーブロックのケースと同様で、またページングの問題をクライアント側で解決しないといけくなるので、ちょっとややこしいいかなと..

いずれにしろ機能としては必要という前提で残しておくということで良いですかね?

@inafact inafact changed the title アカウント削除機能、コミュニティから退会の扱いについて アカウント削除機能、コミュニティからの退会の扱いについて May 13, 2019
@thirozumi
Copy link
Member

基本的には 存在しないユーザー / Unknown User みたい感じに置換かなと思うんですけど、退会したユーザーIDと同一のユーザーIDで作り直せるんであれば、グループメッセージのタイトルとかで変な感じになりそうですね。

@inafact
Copy link
Member Author

inafact commented May 13, 2019

#108 (comment)

上記のグループチャットのデフォルトのタイトル(参加者の名前を「,」でつなげたもの)に関しては、一旦スレッド一覧/詳細で表示する際にAPIレスポンスのタイトルフィールドを利用するのではなく、クライアント側で動的に参加者のユーザー名を参照して生成、という方向にしておきます。

@thirozumi
Copy link
Member

@inafact 逐一生成でもいいけど、もし退会タイミングでグループチャット名を書き換えられたりするのであればサーバーサイドで何とかするのがパフォーマンス的には良さそうという気もしますね。

@hiroyuki どうでしょうか?

@inafact inafact added this to the 1.3 milestone May 24, 2019
@inafact inafact self-assigned this May 24, 2019
@inafact inafact modified the milestones: 1.3, 1.4 Jun 10, 2019
@hiroyuki
Copy link

@thirozumi @inafact すいません、こちらの件、見逃しておりました。
問題は、「存在しないユーザー」としたときに、同一IDで作り直したときに、そのIDで上書きされてしまうということですよね?
サーバー側の仕様も確認しますね。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants