-
Notifications
You must be signed in to change notification settings - Fork 33
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
The Event content type should use Smart Date fields, not datetime_range #69
base: main
Are you sure you want to change the base?
Conversation
Just FYI -- I cannot merge this because it's not installing. This is the error we're getting: https://github.com/phenaproxima/starshot-prototype/actions/runs/9071250306/job/24967409796?pr=69#step:5:791 Probably some misconfiguration in the fields. @mandclu, if you can address this I or another maintainer can manually test and hopefully merge this. |
OK I'll try to have a look tomorrow |
I believe the issue was the field values in the demo content. I added a commit to convert them. |
Looks like there's a bug in the module here, @mandclu...bad enough that it prevents installation. Can you address that? I'll need to at least manually test this before I can merge it. |
I added a commit and then tested the result on a fresh install. The updated recipe worked without issue for me. |
As per the discussion in #64, convert the Event content type to use a Smart Date field, and set the default value to "next hour".