Conversation
Frenzie
left a comment
There was a problem hiding this comment.
Looks straightforward, can't think of any negatives.
Co-authored-by: Frans de Jonge <fransdejonge@gmail.com>
|
Correct me if I'm wrong, but for what I can gather from the code, there is no use of the timezone elsewhere than in the configuration. Do you plan on adding some code to utilize it in the interface or am I missing something? |
See the |
|
I have just added the possibility to pick the timezone when creating a new user:
|
|
@Alkarex I think this PR led to a notice in PHP at each page load: Is this normal? I guess I just have to set a timezone in my PHP conf which has none by default, or select one in FreshRSS conf? |
Add missing tzdata in Docker :newest Fallback to UTC if no timezone is defined at all #fix FreshRSS#4906 (comment)
* Safer timezone set Add missing tzdata in Docker :newest Fallback to UTC if no timezone is defined at all #fix #4906 (comment) * Better refactoring Show fallback timezone everywhere
* Safer timezone set Add missing tzdata in Docker :newest Fallback to UTC if no timezone is defined at all #fix FreshRSS#4906 (comment) * Better refactoring Show fallback timezone everywhere
#fix #2754
Default is server / PHP default, e.g. as defined by
TZin our Docker images, or inphp.inidate.timezone