Skip to content

Use local date time format in CSV#8398

Merged
aembler merged 5 commits intoconcretecms:developfrom
tao-s:patch-1
Feb 14, 2020
Merged

Use local date time format in CSV#8398
aembler merged 5 commits intoconcretecms:developfrom
tao-s:patch-1

Conversation

@tao-s
Copy link
Copy Markdown
Contributor

@tao-s tao-s commented Feb 7, 2020

ATOM format is hard to use for normal user in Japan.

  • I read the guidelines for contributing linked above

  • PHP-only files follow our coding style; in order to do that use php-cs-fixer - http://cs.sensiolabs.org/ - as follows:
    php-cs-fixer fix --config=<webroot>/.php_cs.dist <filename>

ATOM format is hard to use for normal user in Japan.
@tao-s tao-s requested a review from KorvinSzanto February 7, 2020 09:01
@mlocati
Copy link
Copy Markdown
Contributor

mlocati commented Feb 7, 2020

What about making it configurable?

@tao-s
Copy link
Copy Markdown
Contributor Author

tao-s commented Feb 7, 2020

@mlocati like this?
tao-s@f633d7a

ref: concretecms#8398 To configurable datetime format
@mlocati
Copy link
Copy Markdown
Contributor

mlocati commented Feb 7, 2020

@mlocati like this?
tao-s@f633d7a

Yes! 👍

@tao-s
Copy link
Copy Markdown
Contributor Author

tao-s commented Feb 7, 2020

@mlocati But, I think that datetime format is set by locale setting at almost case when use in non-English.
My second commit need configuration in config/concrete.php, not in the dashboard/locale. Is it really useful for all users? I feel it's little bit annoyance now...

tao-s added 2 commits February 8, 2020 15:02
5.6対応しないといかんのか…
@tao-s tao-s requested a review from KorvinSzanto February 10, 2020 07:03
@aembler aembler merged commit 4bf0f59 into concretecms:develop Feb 14, 2020
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