misskey
misskey copied to clipboard
fix: 照会まわりの修正
What
- Fix: ノートの検索欄にURLを入れると勝手に照会される問題を修正
- Fix: ローカルユーザーをローカルのMisskey WebのURL(
/@user
)で照会できない問題を修正
Why
Fix #12832 Fix #12996
Additional info (optional)
意図的にそのURLを含んだノートを検索したい場合に照会が回るとやっぱり困るので、確認ダイアログにした
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
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.
Storybook書かない?
やりかたがわからない
やりかたがわからない
ご希望なら後日インストラクション可
(なお私もわかってない
コンフリクト解消
コンフリクト解消
__ui_user__
というワードを見てもどういうものを指しているのかいまいちピンとこないわね
__local_user__
にしておくか
__
付けてるのはどういう理由かしら
ほかが照会の動作なのに対してコイツだけはユーザーをそのまま取りに行ってるので
うーむ設計がおかしい可能性があるわね
Fix: ローカルユーザーをローカルのMisskey WebのURL(/@user)で照会できない問題を修正
これはフロントエンド側の修正な気がする
いやわからん
いやわからん
ユーザーページのURLで照会リクエストが来ても照会できるべきな気はする(他サーバーはその形式で照会できるので)のでバックエンド
これ逆になぜリモートユーザーの場合はエラーにならないんだろう
parseUri 関数がどういう役目なのかが理解できてない
なぜリモートとローカルの場合で返り値の型が大きく異なるのかしら
/** hint of type, e.g. "notes", "users" */
type: string;
なぜstring? 予めtypeは決まっているものではない?
気付かずに一部 #13858 で被せてしまっていた… (13858ではただconfirmを置いただけなので、こちらのUIの方が分かりやすくて良さそうな感じはします)
意図せず #14128 でこれのレビューにも充分なstoryを書いてしまった可能性がある
PR内容が被ったのとコンフリクト解消が地獄なのでそのうち立て直します