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

Fix date format in anonymous posts #503

Merged
merged 1 commit into from Dec 29, 2021
Merged

Fix date format in anonymous posts #503

merged 1 commit into from Dec 29, 2021

Conversation

StJudeWasHere
Copy link

@StJudeWasHere StJudeWasHere commented Aug 13, 2021

The date format for anonymous posts is always displayed in the default English format.

The anonymous posts SQL query doesn't include the language field which is then used to format the date. As the language field is empty it defaults to the English format.

	loc := monday.FuzzyLocale(p.Language.String)
	res.DisplayDate = monday.Format(p.Created, monday.LongFormatsByLocale[loc], loc)

Adding the language to the anonymous posts query fixes the issue.


  • I have signed the CLA

@thebaer thebaer added this to the 0.13.2 milestone Aug 13, 2021
@thebaer
Copy link
Member

@thebaer thebaer commented Aug 13, 2021

Thanks for finding and fixing this! Will verify and merge this soon.

@thebaer thebaer changed the base branch from develop to hotfix-0.13.2 Sep 29, 2021
@thebaer
Copy link
Member

@thebaer thebaer commented Dec 29, 2021

Thanks again for this. Looks good! Merging now.

@thebaer thebaer merged commit 43ca80f into writefreely:hotfix-0.13.2 Dec 29, 2021
@StJudeWasHere StJudeWasHere deleted the hotfix/fix-date-format branch Jan 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants