- @matthieuscarset opened merge request.
- Status changed to Needs review
almost 2 years ago 1:47am 28 January 2023 - π¨π¦Canada mandclu
Based on my initial testing, this merge request is very promising. With this work applied the recurring events no longer appear more than once. I'll try to do a little more testing before moving this to RTBC.
- Status changed to Needs work
almost 2 years ago 1:58am 28 January 2023 - π¨π¦Canada mandclu
There is some progress, but we're not quite there yet. The work in this branch prevents multiple values from each being displayed for every value, but based on my testing it only ever displays the first value. This is true whether the values are recurring, or simply multiple, non-recurring values.
- π¨π¦Canada mandclu
I did some more testing and realized that this does work as intended as long as the view is configured to show fields instead of rendered content, and the field is configured to not display all values in the same row.
Leaving the issue as "needs work" because the MR looks like it needs to be manually rebased.
- Status changed to Needs review
almost 2 years ago 11:58pm 28 January 2023 TY very much for testing and for your feedback @mandclu @nicxvan
It helped me clarify the issue and I learned more about Smart Date module too π
I made small changes to the processing of View results in this MR and it seems to work correctly - at least from my own tests.
These changes needs more review before I can merge them and release a new version.
Looking forward to hearing from you.
- Status changed to RTBC
almost 2 years ago 2:22pm 29 January 2023 - π¨π¦Canada mandclu
Just tested this again (with the latest commits) and it's still working great for me, with both core multivalued fields and Smart Date recurring values.
In testing I did find something slightly odd (to me, anyway) about the navigation by month pager, but I'll file a separate ticket for that.
IMHO this needs documentation, particularly because it requires a very specific setup to work. Happy to document this in the Smart Date handbook, but it might be worth including in this module's README as well.
-
matthieuscarset β
committed 9a68b5c1 on 1.0.x
Issue #3324673 by matthieuscarset, mandclu, nicxvan: Smart date...
-
matthieuscarset β
committed 9a68b5c1 on 1.0.x
- Status changed to Fixed
almost 2 years ago 4:33pm 29 January 2023 -
matthieuscarset β
committed ef43cf39 on 2.0.x
Fix recurring event duplicate results #3324673
-
matthieuscarset β
committed ef43cf39 on 2.0.x
Automatically closed - issue fixed for 2 weeks with no activity.