Skip to content
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

Enhance(frontend): 年齢を非表示にできるように #12821

Conversation

1STEP621
Copy link
Contributor

What

誕生年に9999年を指定すると、年齢を非表示にできるようにしました。

Why

Resolve #10005

Additional info (optional)

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

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Dec 27, 2023
Copy link

codecov bot commented Dec 27, 2023

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (b46f431) 79.87% compared to head (0e8d180) 79.87%.

Files Patch % Lines
packages/frontend/src/pages/user/home.vue 36.36% 7 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12821      +/-   ##
===========================================
- Coverage    79.87%   79.87%   -0.01%     
===========================================
  Files          956      956              
  Lines       109066   109050      -16     
  Branches      8417     8417              
===========================================
- Hits         87120    87105      -15     
+ Misses       21946    21945       -1     

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

@kakkokari-gtyih
Copy link
Contributor

#10005 (comment)

@1STEP621 1STEP621 marked this pull request as draft December 27, 2023 15:36
@1STEP621 1STEP621 changed the title Enhance(frontend): 誕生年に9999年を指定すると、年齢を非表示にできるように Enhance(frontend): 年齢を非表示にできるように Dec 27, 2023
@1STEP621

This comment was marked as outdated.

@1STEP621 1STEP621 marked this pull request as ready for review December 28, 2023 13:19
@KisaragiEffective
Copy link
Sponsor Member

KisaragiEffective commented Jan 3, 2024

0年にした方が良いのではないかという意見を見かけたので参照用に書いておきますが、西暦0年はグレゴリオ暦でもその前のユリウス暦の妥当な年数ではありません。これはAPI経由で日付をプログラミング言語の暦1として解釈しようとした時に不都合になり得ます。まだ不都合のあるケースは発見できていませんが、わざわざリスキーな選択肢を取る理由はないでしょう。

Footnotes

  1. 世の中には様々な暦がありますが、ここでは1582年以前の日付についてのみ振る舞いが異なるグレゴリオ暦の変種を考えます: A) グレゴリオ暦として遡る B) ユリウス暦として解釈する

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Jan 3, 2024

どちらにせよ元issueでマジックナンバーにすべきかどうかを含め色々検討が進んでいるのでそれを待ったほうが良さそう

@1STEP621
Copy link
Contributor Author

1STEP621 commented Jan 3, 2024

Firefox, Chromeともに、そもそもブラウザ側で0000年を入力することはできませんでした
↓Chromeの例(0000を入力しているが、勝手に0001にバリデーションされている)

Screencast.from.2024.01.03.22.32.23.webm

Copy link
Contributor

github-actions bot commented Jan 3, 2024

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

差分はこちら

Get diff files from Workflow Page

@1STEP621
Copy link
Contributor Author

元issueでもう少し議論が進んでからのほうが良さそうなので一旦closeします🙏

@1STEP621 1STEP621 closed this Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend:test packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR packages/misskey-js
Projects
None yet
Development

Successfully merging this pull request may close these issues.

誕生日に月日のみ設定したい
3 participants