-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix: 照会まわりの修正 #13342
fix: 照会まわりの修正 #13342
Conversation
このPRによるapi.jsonの差分 差分はこちら |
Codecov ReportAttention: Patch coverage is
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 ☔ View full report in Codecov by Sentry. |
Storybook書かない? |
やりかたがわからない |
ご希望なら後日インストラクション可 |
(なお私もわかってない |
コンフリクト解消 |
コンフリクト解消 |
|
|
|
ほかが照会の動作なのに対してコイツだけはユーザーをそのまま取りに行ってるので |
うーむ設計がおかしい可能性があるわね |
これはフロントエンド側の修正な気がする |
いやわからん |
ユーザーページのURLで照会リクエストが来ても照会できるべきな気はする(他サーバーはその形式で照会できるので)のでバックエンド |
これ逆になぜリモートユーザーの場合はエラーにならないんだろう |
parseUri 関数がどういう役目なのかが理解できてない |
なぜリモートとローカルの場合で返り値の型が大きく異なるのかしら |
なぜstring? |
気付かずに一部 #13858 で被せてしまっていた… |
意図せず #14128 でこれのレビューにも充分なstoryを書いてしまった可能性がある |
PR内容が被ったのとコンフリクト解消が地獄なのでそのうち立て直します |
What
/@user
)で照会できない問題を修正Why
Fix #12832
Fix #12996
Additional info (optional)
意図的にそのURLを含んだノートを検索したい場合に照会が回るとやっぱり困るので、確認ダイアログにした
Checklist