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

drop messaging #9919

Merged
merged 3 commits into from
Feb 15, 2023
Merged

drop messaging #9919

merged 3 commits into from
Feb 15, 2023

Conversation

syuilo
Copy link
Member

@syuilo syuilo commented Feb 13, 2023

What

Why

Additional info (optional)

@github-actions github-actions bot added packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR 🧪Test labels Feb 13, 2023
@syuilo
Copy link
Member Author

syuilo commented Feb 13, 2023

既存のチャットUI活かそうかと思ってたけど、どうせノートにするならリアクションとかもしたいしそう考えるとMkNoteそのまま使いたいからチャット関係のUIも削除かしらね

@codecov
Copy link

codecov bot commented Feb 13, 2023

Codecov Report

Merging #9919 (ce3e0da) into develop (c1f1e0e) will increase coverage by 0.19%.
The diff coverage is 45.45%.

@@             Coverage Diff             @@
##           develop    #9919      +/-   ##
===========================================
+ Coverage    22.89%   23.09%   +0.19%     
===========================================
  Files          729      717      -12     
  Lines        68012    66302    -1710     
  Branches      2069     2021      -48     
===========================================
- Hits         15570    15311     -259     
+ Misses       52442    50991    -1451     
Impacted Files Coverage Δ
packages/backend/src/core/CoreModule.ts 100.00% <ø> (ø)
packages/backend/src/core/GlobalEventService.ts 68.31% <ø> (+0.31%) ⬆️
...ckages/backend/src/core/PushNotificationService.ts 34.78% <0.00%> (-0.76%) ⬇️
.../backend/src/core/activitypub/ApRendererService.ts 31.50% <0.00%> (+<0.01%) ⬆️
packages/backend/src/core/activitypub/type.ts 88.25% <ø> (-0.04%) ⬇️
packages/backend/src/di-symbols.ts 100.00% <ø> (ø)
packages/backend/src/misc/schema.ts 0.00% <ø> (ø)
packages/backend/src/models/index.ts 100.00% <ø> (ø)
packages/backend/src/models/schema/user.ts 0.00% <ø> (ø)
packages/backend/src/postgres.ts 93.44% <ø> (-0.06%) ⬇️
... and 13 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@EbiseLutica
Copy link
Member

既存のチャットはダイレクト投稿にマイグレーションできたほうが良さそう データとして残しておきたい

@tamaina
Copy link
Member

tamaina commented Feb 14, 2023

既存のチャットUI活かそうかと思ってたけど、どうせノートにするならリアクションとかもしたいしそう考えるとMkNoteそのまま使いたい

チャットUIにMkNoteやMkPostFormそのまま突っ込めないかしら

既存のチャットはダイレクト投稿にマイグレーションできたほうが

しないのか(Misskeyのことだからしないと言うのもありそうだとは思った)

@tamaina
Copy link
Member

tamaina commented Feb 14, 2023

まずノートにグループを紐づけられるようにする必要があるか

@EbiseLutica
Copy link
Member

ダイレクトの送信先は複数選べるから、とりあえずグループメンバー全員突っ込むとかでもいいのかなーとは思った ただ他にも結構めんどくさいことをする必要があるから、難しいかも

@tamaina
Copy link
Member

tamaina commented Feb 14, 2023

グループメンバー全員突っ込む

もしチャットルームUIを維持するならグループがないと破綻する気がする

@EbiseLutica
Copy link
Member

そのへんってグループ廃止された現在ってどうなってるんだっけ

@tamaina
Copy link
Member

tamaina commented Feb 14, 2023

グループはUIから削除されただけ

@tamaina
Copy link
Member

tamaina commented Feb 14, 2023

なんかでもメンテナンスとか考えるとグループもチャットUIもなかったことにした方がいい気もしなくもない

@EbiseLutica
Copy link
Member

EbiseLutica commented Feb 14, 2023

まーさらっとけしちゃうか

@syuilo
Copy link
Member Author

syuilo commented Feb 14, 2023

もしチャットルームUIを維持するならグループがないと破綻する気がする

そう?

@tamaina
Copy link
Member

tamaina commented Feb 14, 2023

どの会話の宛先をグループとするかが不確定すぎない?
スレッドだと分岐があるし(外との互換性を考えなければいける気がしてきたけどそんなことできるかな)

@syuilo
Copy link
Member Author

syuilo commented Feb 14, 2023

どの会話の宛先をグループとするか

単に同じ宛先は同じグループとして扱うのを想定してた

Aに対するメッセージは常にグループA、A/B/Cに対するメッセージは常にグループB

@EbiseLutica
Copy link
Member

暗黙的にグループのマイグレも兼ねる感じか

@tamaina
Copy link
Member

tamaina commented Feb 14, 2023

ユーザーの増減があるとひっちゃかめっちゃかになる気がした

@tamaina
Copy link
Member

tamaina commented Feb 14, 2023

(ノートの仕様的に減ることはないのか?

@syuilo
Copy link
Member Author

syuilo commented Feb 14, 2023

途中からユーザー増減するケースそんなにない気がするし、ひっちゃかめっちゃかになっても(途中からグループが変わっても)そこまで困ることはなさそうに思った

@tamaina
Copy link
Member

tamaina commented Feb 14, 2023

大規模なグループになると頻繁に出たり入ったりがありそう

(そもそもダイレクトって自分から出られない気がするんだけど)

@syuilo
Copy link
Member Author

syuilo commented Feb 15, 2023

とりあえずこのPRでは消すだけで

@syuilo syuilo merged commit 8f2049b into develop Feb 15, 2023
@syuilo syuilo deleted the drop-messaging branch February 15, 2023 04:06
ltlapy added a commit to ltlapy/misskey that referenced this pull request Feb 18, 2023
digitalsp added a commit to digitalsp/misskey-for-moge that referenced this pull request Mar 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants