-
Notifications
You must be signed in to change notification settings - Fork 34
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
Added support for nil date input. #56
base: main
Are you sure you want to change the base?
Conversation
Bump |
1 similar comment
Bump |
@saboter hey hey, apologies for not getting back here... this is on my radar, just haven't had time lately but I'll be taking a look soon. Thanks. |
@carlosantoniodasilva Hello. Good news, thank you. I have enabled rails 6.1 in my fork in between which most probably broke the CI tests. |
@saboter right, thanks for that. I'm looking into those right now 👍 |
The fork is deployed in production for couple of days with newest Rails and so far no issues. |
Bump :). |
Bump. |
Would be nice if the localized assign did not fail on nil date input.
E.g. object.localized({date_col: nil}) throws now:
no implicit conversion of nil into String
# /Users/work/.rvm/gems/ruby-2.5.8/gems/i18n_alchemy-0.3.1/lib/i18n_alchemy/date_parser.rb:9:in `_strptime'