misskey
misskey copied to clipboard
fix(frontend): 「今日誕生日のフォロー中ユーザー」ウィジェットが正しく動作しない問題を修正
Fix https://github.com/misskey-dev/misskey/pull/12835
What
- 誕生日を問い合わせるパラメーターを、ISO8601形式からbirthdaySchema(
yyyy-mm-dd
)に変更 - ウィジェットの名前を誤解のないように変更
- マニュアルでリロードできるようにした
Why
えー、申し訳ございません 今日誕生日のフォロー中を表示する機能として実装したのですがやはり実装がおかしくて昨日誕生日の人を表示する機能になってしまいました
Fix https://github.com/misskey-dev/misskey/pull/12450#discussion_r1431365294
Additional info (optional)
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 66.66667%
with 1 lines
in your changes are missing coverage. Please review.
Project coverage is 63.21%. Comparing base (
8f415d6
) to head (24c747b
).
:exclamation: Current head 24c747b differs from pull request most recent head 69aaa39. Consider uploading reports for the commit 69aaa39 to get more accurate results
Files | Patch % | Lines |
---|---|---|
...ackend/src/server/api/endpoints/users/following.ts | 66.66% | 1 Missing :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## develop #12835 +/- ##
===========================================
- Coverage 66.20% 63.21% -3.00%
===========================================
Files 990 986 -4
Lines 115894 111271 -4623
Branches 4477 5463 +986
===========================================
- Hits 76729 70341 -6388
- Misses 39133 39261 +128
- Partials 32 1669 +1637
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
このPRによるapi.jsonの差分
差分はこちら
--- base
+++ head
@@ -66122,7 +66122,8 @@
"type": [
"string",
"null"
- ]
+ ],
+ "pattern": "^([0-9]{4})-([0-9]{2})-([0-9]{2})$"
}
},
"anyOf": [
パフォーマンス的な観点と今後の活用のためにMM/DD/YYYY
形式でデータを変換したバージョンのインデックスを張っておくといいかも
複雑な処理がなくてもLIKE%クエリーですぐ出てくるようになるはずなので
もしくはGINインデックスを張ると後方一致の%LIKEクエリーでもインデックスが使えるので参考にしてください
できればこのついでにbirthdayコラムのタイプをdateに変えちゃいたい気もしますが…
😮
https://github.com/misskey-dev/misskey/issues/10005 での議論の結論によっては誕生日のスキーマが変更される可能性があるのでまだマージできない
#10005 での議論の結論によっては誕生日のスキーマが変更される可能性があるのでまだマージできない
方針決定まで時間かかりそうなので一旦マージしてしまっても良いかもしれない
#10005 での議論の結論によっては誕生日のスキーマが変更される可能性があるのでまだマージできない
方針決定まで時間かかりそうなので一旦マージしてしまっても良いかもしれない
👀
そう思う
コンフリクト解消
👀
ウィジェットのタイトルはもう少し短くしないと場所によっては途切れそう
元のままで行くか…
文言の変更は取消した
👍🏻