misskey icon indicating copy to clipboard operation
misskey copied to clipboard

fix: 照会まわりの修正

Open kakkokari-gtyih opened this issue 1 year ago • 20 comments

What

  • Fix: ノートの検索欄にURLを入れると勝手に照会される問題を修正
  • Fix: ローカルユーザーをローカルのMisskey WebのURL(/@user)で照会できない問題を修正

Why

Fix #12832 Fix #12996

Additional info (optional)

意図的にそのURLを含んだノートを検索したい場合に照会が回るとやっぱり困るので、確認ダイアログにした

image

Checklist

  • [x] Read the contribution guide
  • [ ] Test working in a local environment
  • [ ] (If needed) Add story of storybook
  • [x] (If needed) Update CHANGELOG.md
  • [ ] (If possible) Add tests

kakkokari-gtyih avatar Feb 16 '24 12:02 kakkokari-gtyih

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

github-actions[bot] avatar Feb 16 '24 12:02 github-actions[bot]

Codecov Report

Attention: Patch coverage is 0% with 20 lines in your changes are missing coverage. Please review.

Project coverage is 64.94%. Comparing base (2b8056a) to head (61a00e3).

Files Patch % Lines
...ackend/src/core/activitypub/ApDbResolverService.ts 0.00% 15 Missing :warning:
.../backend/src/core/activitypub/ApResolverService.ts 0.00% 5 Missing :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #13342      +/-   ##
===========================================
- Coverage    64.95%   64.94%   -0.02%     
===========================================
  Files          989      989              
  Lines       112848   112868      +20     
  Branches      5770     4484    -1286     
===========================================
  Hits         73306    73306              
- Misses       38100    39531    +1431     
+ Partials      1442       31    -1411     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Feb 16 '24 12:02 codecov[bot]

Storybook書かない?

tamaina avatar Feb 16 '24 13:02 tamaina

やりかたがわからない

kakkokari-gtyih avatar Feb 16 '24 13:02 kakkokari-gtyih

やりかたがわからない

ご希望なら後日インストラクション可

acid-chicken avatar Feb 16 '24 13:02 acid-chicken

(なお私もわかってない

tamaina avatar Feb 16 '24 13:02 tamaina

コンフリクト解消

kakkokari-gtyih avatar Mar 04 '24 07:03 kakkokari-gtyih

コンフリクト解消

kakkokari-gtyih avatar Apr 27 '24 08:04 kakkokari-gtyih

__ui_user__というワードを見てもどういうものを指しているのかいまいちピンとこないわね

syuilo avatar Apr 27 '24 08:04 syuilo

__local_user__ にしておくか

kakkokari-gtyih avatar Apr 27 '24 08:04 kakkokari-gtyih

__付けてるのはどういう理由かしら

syuilo avatar Apr 27 '24 09:04 syuilo

ほかが照会の動作なのに対してコイツだけはユーザーをそのまま取りに行ってるので

kakkokari-gtyih avatar Apr 27 '24 09:04 kakkokari-gtyih

うーむ設計がおかしい可能性があるわね

syuilo avatar Apr 27 '24 09:04 syuilo

Fix: ローカルユーザーをローカルのMisskey WebのURL(/@user)で照会できない問題を修正

これはフロントエンド側の修正な気がする

syuilo avatar Apr 27 '24 09:04 syuilo

いやわからん

syuilo avatar Apr 27 '24 09:04 syuilo

いやわからん

ユーザーページのURLで照会リクエストが来ても照会できるべきな気はする(他サーバーはその形式で照会できるので)のでバックエンド

kakkokari-gtyih avatar Apr 27 '24 09:04 kakkokari-gtyih

これ逆になぜリモートユーザーの場合はエラーにならないんだろう

syuilo avatar Apr 27 '24 09:04 syuilo

parseUri 関数がどういう役目なのかが理解できてない

syuilo avatar Apr 27 '24 09:04 syuilo

なぜリモートとローカルの場合で返り値の型が大きく異なるのかしら

syuilo avatar Apr 27 '24 09:04 syuilo

	/** hint of type, e.g. "notes", "users" */
	type: string;

なぜstring? 予めtypeは決まっているものではない?

syuilo avatar Apr 27 '24 09:04 syuilo

気付かずに一部 #13858 で被せてしまっていた… (13858ではただconfirmを置いただけなので、こちらのUIの方が分かりやすくて良さそうな感じはします)

Sayamame-beans avatar Jul 30 '24 12:07 Sayamame-beans

意図せず #14128 でこれのレビューにも充分なstoryを書いてしまった可能性がある

tai-cha avatar Jul 30 '24 23:07 tai-cha

PR内容が被ったのとコンフリクト解消が地獄なのでそのうち立て直します

kakkokari-gtyih avatar Jul 31 '24 10:07 kakkokari-gtyih