misskey icon indicating copy to clipboard operation
misskey copied to clipboard

fix(frontend): 「今日誕生日のフォロー中ユーザー」ウィジェットが正しく動作しない問題を修正

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

Fix https://github.com/misskey-dev/misskey/pull/12835

What

  • 誕生日を問い合わせるパラメーターを、ISO8601形式からbirthdaySchema(yyyy-mm-dd)に変更
  • ウィジェットの名前を誤解のないように変更
  • マニュアルでリロードできるようにした image

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

kakkokari-gtyih avatar Dec 28 '23 08:12 kakkokari-gtyih

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.

codecov[bot] avatar Dec 28 '23 08:12 codecov[bot]

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

差分はこちら
--- base
+++ head
@@ -66122,7 +66122,8 @@
                     "type": [
                       "string",
                       "null"
-                    ]
+                    ],
+                    "pattern": "^([0-9]{4})-([0-9]{2})-([0-9]{2})$"
                   }
                 },
                 "anyOf": [

Get diff files from Workflow Page

github-actions[bot] avatar Dec 28 '23 08:12 github-actions[bot]

パフォーマンス的な観点と今後の活用のためにMM/DD/YYYY形式でデータを変換したバージョンのインデックスを張っておくといいかも 複雑な処理がなくてもLIKE%クエリーですぐ出てくるようになるはずなので もしくはGINインデックスを張ると後方一致の%LIKEクエリーでもインデックスが使えるので参考にしてください できればこのついでにbirthdayコラムのタイプをdateに変えちゃいたい気もしますが…

u1-liquid avatar Jan 08 '24 08:01 u1-liquid

😮

kakkokari-gtyih avatar Jan 24 '24 07:01 kakkokari-gtyih

https://github.com/misskey-dev/misskey/issues/10005 での議論の結論によっては誕生日のスキーマが変更される可能性があるのでまだマージできない

kakkokari-gtyih avatar Jan 26 '24 00:01 kakkokari-gtyih

#10005 での議論の結論によっては誕生日のスキーマが変更される可能性があるのでまだマージできない

方針決定まで時間かかりそうなので一旦マージしてしまっても良いかもしれない

kakkokari-gtyih avatar Feb 06 '24 10:02 kakkokari-gtyih

#10005 での議論の結論によっては誕生日のスキーマが変更される可能性があるのでまだマージできない

方針決定まで時間かかりそうなので一旦マージしてしまっても良いかもしれない

👀

kakkokari-gtyih avatar Feb 15 '24 11:02 kakkokari-gtyih

そう思う

1Step621 avatar Feb 15 '24 11:02 1Step621

コンフリクト解消

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

👀

kakkokari-gtyih avatar Mar 12 '24 03:03 kakkokari-gtyih

ウィジェットのタイトルはもう少し短くしないと場所によっては途切れそう

syuilo avatar Mar 20 '24 08:03 syuilo

元のままで行くか…

kakkokari-gtyih avatar Mar 22 '24 08:03 kakkokari-gtyih

文言の変更は取消した

kakkokari-gtyih avatar Mar 25 '24 09:03 kakkokari-gtyih

👍🏻

syuilo avatar Mar 25 '24 09:03 syuilo