- Issue created by @joevagyok
- last update
about 1 year ago 157 pass, 4 fail - @joevagyok opened merge request.
- last update
about 1 year ago 162 pass - Status changed to Needs review
about 1 year ago 10:05am 4 October 2023 - last update
about 1 year ago 162 pass - π§πͺBelgium joevagyok
Uploading a patch version of the merge request for composer patching.
- First commit to issue fork.
- last update
about 1 year ago 162 pass - πΊπΈUnited States dcam
For what it's worth, I can't reproduce this issue of the front page crashing. I think it might be implementation-specific. That might explain how I've been developing on a site nearly every day for weeks with this update on it without experiencing that particular problem.
I made some changes to the MR branch. I'm going to try to write a test for the new update function. I don't need another thing breaking peoples' sites just because I didn't test it.
Also, thank you both for working on this.
- π§πͺBelgium joevagyok
On our builds/sites, after the update path deleted those values including the path from the view, the empty path pointed to "/" the homepage and that messed up the route.
I agree with the changes pointed out above, thanks. The MR looks good for me, I will reupload the patch based on the latest changes.
- last update
about 1 year ago 162 pass - π§πͺBelgium joevagyok
Adding a patch based on the latest changes of the MR.
- last update
about 1 year ago 163 pass - last update
about 1 year ago 163 pass - last update
about 1 year ago 163 pass -
dcam β
committed 3a8efc82 on 2.x authored by
joevagyok β
Issue #3391559: Update views.view.aggregator_rss_feed update path resets...
-
dcam β
committed 3a8efc82 on 2.x authored by
joevagyok β
- last update
about 1 year ago 163 pass - Status changed to Fixed
about 1 year ago 1:59am 6 October 2023 - πΊπΈUnited States dcam
Thank you for the patch and reviews. I appreciate it.
Automatically closed - issue fixed for 2 weeks with no activity.