Skip to content

fix(frontend): 誕生日の日付が言語ごとのフォーマットで表示されるように#17105

Draft
kakkokari-gtyih wants to merge 2 commits intomisskey-dev:developfrom
kakkokari-gtyih:fix-17084
Draft

fix(frontend): 誕生日の日付が言語ごとのフォーマットで表示されるように#17105
kakkokari-gtyih wants to merge 2 commits intomisskey-dev:developfrom
kakkokari-gtyih:fix-17084

Conversation

@kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Jan 17, 2026

What

言語ごとのフォーマットで誕生日の日付表記を表示するように

image image

Why

Fix #17084

Additional info (optional)

frontend側のintl-constは使われてなかったので消した

Checklist

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

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jan 17, 2026
@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jan 17, 2026
@codecov
Copy link

codecov bot commented Jan 17, 2026

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 14.04%. Comparing base (389861f) to head (4597313).

Files with missing lines Patch % Lines
packages/frontend/src/pages/user/home.vue 75.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #17105       +/-   ##
============================================
- Coverage    63.65%   14.04%   -49.62%     
============================================
  Files         1160      240      -920     
  Lines       115616    11362   -104254     
  Branches      8168     3765     -4403     
============================================
- Hits         73598     1596    -72002     
+ Misses       39834     7623    -32211     
+ Partials      2184     2143       -41     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@anatawa12
Copy link
Member

USの人に聞いて確認するべきな気がするけど、1/17/2026Jan 17, 2026のどっちがいいか要検討な気がする

(USとUKがnumeric-numeric-numericだと'1/17/2026'(US)と'17/01/2026'(UK)で区別がつきづらいことについてUS/UKの人がどう思ってるかによってはnumeric-short-numericの'Jan 17, 2026'(US)'17 Jan 2026'(UK)'2026年1月17日'(JP)のほうが良かったりしそうだなと思った)

@kakkokari-gtyih
Copy link
Contributor Author

とりあえず既存のフォーマットスタイルに合わせました
表記法についてはスコープ外そう

@anatawa12
Copy link
Member

JSのdatetimeの範囲外の値を取れる(2月31日等)ので、それの対応が漏れてるかも

@anatawa12
Copy link
Member

とりあえず既存のフォーマットスタイルに合わせました

既存の場合(ISO等と同じ)YYYY MM DDだから区別がつかないことがないけど言語別にすると区別がつかなくなる可能性が出てくるので合わせたと一概言えるかは怪しいような気もする...

@kakkokari-gtyih
Copy link
Contributor Author

多分問題ないはず

image

@anatawa12
Copy link
Member

もともと2月31日と表現されてたのが3月になってユーザにとって意図した日時になってない

@kakkokari-gtyih
Copy link
Contributor Author

2月31日と表現されてた

これそのものがバグじゃないかしら
仕様とされたっけ?

@anatawa12
Copy link
Member

少なくともユーザが期待した動作ではないなと。
フロントで再解釈するようにするならちゃんとバックエンドでエラーで設定できなくする変更と同時に行うべきかなと私は思う

@anatawa12
Copy link
Member

anatawa12 commented Jan 17, 2026

多分問題ないはず
image

JSCoreはだめそう。
V8とspidermonkeyはimplementation-defined date formatとして対応してるっぽい

image

@kakkokari-gtyih kakkokari-gtyih marked this pull request as draft January 17, 2026 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packages/frontend Client side specific issue/PR size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Development

Successfully merging this pull request may close these issues.

Birthday date is not displayed in local format

2 participants