- 🇮🇳India Chandra Gowsalya Kannan
Tested Patch #2 which works good and make D10 compatibility. But found some drupal check errors which i attached for reference.
I have resolved that drupal check errors in my patch and tested in Drupal 9.5.7 and Drupal 10 - Status changed to Needs review
over 1 year ago 1:54pm 19 April 2023 - @loopy1492 opened merge request.
- Status changed to RTBC
over 1 year ago 6:33am 24 April 2023 - 🇮🇳India NishaVaghela
I have reviewed the patch in comment #6 , it is getting applied successfully.
Also, there are no compatibility errors found , thus moving the issue to RTBC. - 51e3bad3 committed on 8.x-1.x
Issue #3290466 by loopy1492, Chandra Gowsalya Kannan, shaunole:...
- 51e3bad3 committed on 8.x-1.x
- Status changed to Fixed
over 1 year ago 2:07pm 31 July 2023 - 🇬🇧United Kingdom Eli-T Manchester
Thanks everyone who tested this issue.
I have committed the patch in #3290466-2: Automated Drupal 10 compatibility fixes → to keep this commit focused only on the Drupal 10 compatibility rather than the other issues addressed by the patch in [#3290466#6].
@loopy1492 - there is no need for or benefit to uploading patches from merge requests; you can retrieve a .patch file by adding .patch to the end of the URL of the merge request.
For example, for the merge request in this issue, change
https://git.drupalcode.org/project/views_ef_fieldset/-/merge_requests/9
to
https://git.drupalcode.org/project/views_ef_fieldset/-/merge_requests/9....
Automatically closed - issue fixed for 2 weeks with no activity.