-
Notifications
You must be signed in to change notification settings - Fork 7
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
UTC Query #230
Comments
@Dan-613 maybe It's not correct. But this don't solve every problem because there is people in a lot of time zones, I need to take a time to work on it and find the best approach. |
I think the only way around that is to ask the user for their timezone and then handle all date validation using that timezone. For a user in Australia not to be able to enter today's date becuase it's not today in UTC, they would be prevented from using the system. The alternative is using something like moment.js to handle conversion between local timezone and UTC but I've not properly thought that through yet and might not be possible since we are ignoring the time of day. |
Yes, maybe this is the best approach, ask for user in the first login as first and last name. I'm thinking in ask for currency in this screen also, then we can configure application to format all currency in application. I'll take some time to plan and write issues for this tomorrow. |
Is this correct? Converting to UTC? What happens if the user is not UTC-based?
EconoFlow/EasyFinance.Domain/Financial/BaseFinancial.cs
Line 57 in d8f1a57
The text was updated successfully, but these errors were encountered: