Skip to content
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

Moving from date to date range in search widget causes fatal error #2115

Closed
omarkasem opened this issue Aug 18, 2024 · 6 comments · Fixed by #2116
Closed

Moving from date to date range in search widget causes fatal error #2115

omarkasem opened this issue Aug 18, 2024 · 6 comments · Fixed by #2116
Assignees

Comments

@omarkasem
Copy link
Collaborator

Moving from date to date range in search widget causes fatal error to recreate it:

@omarkasem omarkasem self-assigned this Aug 18, 2024
@omarkasem omarkasem linked a pull request Aug 18, 2024 that will close this issue
@mrcasual
Copy link
Collaborator

@omarkasem, we were unable to reproduce the issue using develop on GravityView and issue/53-support-gravityviews-search-bar2 on Event Field. Could you please record a video or reproduce it on https://site.try.gravitykit.com/?

@rafaehlers
Copy link
Contributor

I was able to reproduce it and the PR fixed it!

@mrcasual
Copy link
Collaborator

I was able to reproduce it and the PR fixed it!

@rafaehlers, which WP/PHP/GV versions are you using? Did you follow @omarkasem's steps? Any other suggestions for reproducing this consistently, given that both @Mwalek and I were unsuccessful?

@rafaehlers
Copy link
Contributor

@mrcasual I followed Omar's steps precisely as he described it.

@mrcasual
Copy link
Collaborator

mrcasual commented Aug 21, 2024

@rafaehlers, OK, since you were able to reproduce it, please come up with a changelog entry.

@rafaehlers
Copy link
Contributor

@mrcasual

Fixed: Fatal Error that happened when a date search is performed against incorrect date fields.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants