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

現状のロジックだとコントリビューターが後ろから40人目以降の絵文字が表示されない問題 #279

Open
1 task
SEKI-YUTA opened this issue Aug 19, 2024 · 1 comment

Comments

@SEKI-YUTA
Copy link
Contributor

SEKI-YUTA commented Aug 19, 2024

現在の状態

現在のsrc/utils/contributors-grouping.tsのロジックだと逆から数えて40人目以降がContributors.jsonに入力した絵文字が表示されない
https://github.com/first-contributions-ja/first-contributions-ja.github.io/blob/main/src/utils/contributors-grouping.ts

理想の状態

40人目以降が入力した絵文字も表示される

タスク一覧

  • ランダムで40人を抽出する or 配列からコントリビュータを取り出す際のオフセットをランダムにする

その他

自分がissueを投げておきながらなのですが、40人目以降の絵文字が反映されていなくても特段支障がないと思います。
コードを読んでいて気になったので立てさせていただきました。

@SEKI-YUTA SEKI-YUTA changed the title 現状のロジックだとコントリビューターが40人目以降の絵文字が表示されない問題 現状のロジックだとコントリビューターが後ろから40人目以降の絵文字が表示されない問題 Aug 19, 2024
@pss-aileen
Copy link
Collaborator

@SEKI-YUTA

こんにちは!

おっしゃるとおり、今まさに40人を超えており表示されない方が出てきました。

40人目以降の表示方法については方針が定まっていませんが、
ぼんやりと全員分をなんとか表示したい...という思いが一個人としてあります。

メンター勢と提案していただいた内容を検討させていただきます!
ありがとうございます🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants