-
-
Notifications
You must be signed in to change notification settings - Fork 685
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
"Experience" trophy score threshold adjustments #260
base: master
Are you sure you want to change the base?
Conversation
Adjusted a few of the score thresholds for the AccountDurationTrophy to be more accurate estimates of the durations in the code comments.
@TurtleCode84 is attempting to deploy a commit to the ryo-ma's Team Team on Vercel. A member of the Team first needs to authorize it. |
10 years is approximately 3652 to 3653 days and 20 years is approximately 7304 to 7305 days. |
Right; in terms of math, I think I did something like Same concept for the 20 years. |
@AlexcastroDev Requesting review and any suggestions for improvement. |
Hi, Thanks for pinging me and taking time to improve the project :) I have two points in this kind of solution:
If you want to do it, go ahead. |
@TurtleCode84 I understand the argument. github-profile-trophy/src/user_info.ts Lines 87 to 89 in 7e5ec49
It's my fault for merging the above code. |
@bhavberi I feel that there is no need to calculate DurationDays by shortening it to 100 days. |
Adjusted two of the score thresholds for the
AccountDurationTrophy
to be more accurate estimates of the durations in the code comments."Seasoned Veteran": 70 changed to 73 (20 years)
"Master Dev": 40 changed to 37 (10 years)
See #258 for additional discussion