-
Notifications
You must be signed in to change notification settings - Fork 24
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
dt.render.date should check for a null value #66
Comments
This is in line with the default renderer. What would you want it to do/render in case of missing date? Do you think the behavior should be configurable (e.g. a 'fallback text' argument)? I am also open for a pull request in this matter. |
Well so here's one thing I found out. If you are storing the data in the DATE format it is returned as a DATE relative to UTC even if you override the server timezone and the database timezone. I do think the fallback text would be a good option. Below is a sample of what my custom render is.
|
Yes, your observation is correct. What I don't understand is why you use custom parsing of the string in the first method. Would you like to prepare a pull request with the fallback mechanics? |
When using the provided dt.render.date if the date passed is null an error will occur.
The text was updated successfully, but these errors were encountered: