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

Leaderboard dates using format and language #3183

Conversation

sulikdan
Copy link
Contributor

Simple fix for an issue - #3176

Description

Took simpler approach, there are still functions that could be also refactored(DateTime), but would require more boiler plate code of how should specific format look like. Moreover Added support only for Short, Medium and Long style of Date

Test

  • Created a few simple test - really not covering all cases, as there are ton of combination
  • Manual tested

Copy link
Member

@Sheikah45 Sheikah45 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be simplified by just using a switch statement,

Also if we no longer use the locale in date info it should be removed.

@sulikdan sulikdan requested a review from Sheikah45 May 29, 2024 12:27
@Sheikah45 Sheikah45 enabled auto-merge (squash) May 29, 2024 22:23
@Sheikah45 Sheikah45 disabled auto-merge May 29, 2024 22:23
@Sheikah45 Sheikah45 enabled auto-merge (squash) May 29, 2024 22:24
Comment on lines +3 to +4
<option name="ALTERNATIVE_JRE_PATH" value="temurin-21" />
<option name="ALTERNATIVE_JRE_PATH_ENABLED" value="true" />
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this supposed to be here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It just specifies the jre for the run config if someone doesn't have it set

@Sheikah45 Sheikah45 merged commit 14f6466 into FAForever:develop May 29, 2024
2 checks passed
@sulikdan sulikdan deleted the leaderboard-dates-using-format-and-language branch May 30, 2024 18:48
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

Successfully merging this pull request may close these issues.

4 participants