- Issue created by @lolmanKD
- Status changed to Needs review
3 months ago 4:50pm 27 May 2024 - Status changed to Needs work
3 months ago 5:26pm 27 May 2024 - ๐บ๐ธUnited States smustgrave
Thanks for reporting
Have not reviewed but issue summary needs to be updated to use full template.
Will need test coverage.Patch should be converted to an MR against 11.x as the latest development branch.
- @kedarkhaire opened merge request.
- ๐ฎ๐ณIndia lolmanKD
Hi @smustgrave,
Can you tell me what kind of test coverage you will require for the above attached patch/Merge Request ?
I tried to add test for the patch but it shows support only till D10.1 and not 10.2 & 11.x
Thanks!
- ๐ณ๐ฟNew Zealand quietone New Zealand
Fixes are made on on 11.x (our main development branch) first, and are then back ported as needed according to our policies.
- ๐ฎ๐ณIndia samit.310@gmail.com
samit.310@gmail.com โ made their first commit to this issueโs fork.
- @samit310gmailcom opened merge request.
- ๐ฎ๐ณIndia samit.310@gmail.com
samit.310@gmail.com โ changed the visibility of the branch 3450175-typeerror-fieldtypepluginmanagercreatefielditem-called to hidden.
- Status changed to Needs review
about 1 month ago 9:46am 26 July 2024 - Status changed to Needs work
about 1 month ago 4:05pm 28 July 2024 - ๐บ๐ธUnited States smustgrave
For the issue summary update and test coverage
I tried to add test for the patch but it shows support only till D10.1 and not 10.2 & 11.x
@kedarkhaire not sure I follow. Can add the test coverage directly to the MR.
- ๐ฎ๐ณIndia lolmanKD
Hi @smustgrave & @samit.310@gmail.com We have found other way to solve it without hampering the core module code.
So now we are good to close this.
Adding changes here, if they face such issue, they can make similar changes to their codebase.
Here is the fix we applied - https://github.com/apigee/apigee-edge-drupal/pull/1076/filesThanks!
- Status changed to Fixed
16 days ago 9:29am 23 August 2024 - ๐ฎ๐ณIndia lolmanKD
Also major credit and thanks to @shishirsuvarna for the fix.
Thanks! Automatically closed - issue fixed for 2 weeks with no activity.