-
Notifications
You must be signed in to change notification settings - Fork 124
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
Corrects the sharing of predicates between date_created and created_at. #6866
Conversation
Test Results 17 files + 1 17 suites +1 2h 17m 33s ⏱️ + 11m 55s Results for commit 9c3be46. ± Comparison against base commit 4b74958. This pull request removes 257 and adds 396 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
@bwatson78 code seems good there. My question to you is what kind of change is it? If you have existing records, does changing this predicate break things? Specifically if I have data in created at and then change the predicate, what happens the next time I open that record? Do we need a data migration of some sort here? |
Summary
Corrects the sharing of predicates between
date_created
andcreated_at
.Type of change (for release notes)
notes-valkyrie
Valkyrie Progress@samvera/hyrax-code-reviewers