- last update
12 months ago 1 pass, 2 fail - last update
12 months ago 2 pass - πΊπΈUnited States erutan
#17 applies cleanly to 2.4 and works great.
While manual entry would work for very small bits of info, and having the file uploads is great for more complex use cases, in ours we need to add in data produced by another program - copy and pasting is a lot simpler than generating a file and then attaching it. I'd recommend this get patched in. :)
- πΊπΈUnited States erutan
Fix formatting in patch on permissions.yml to match earlier code cleanup so it applies to dev.
- πΊπΈUnited States erutan
Tests give the following new errors:
Drupal.Commenting.FunctionComment.ParamMissingDefinition at /builds/project/tablefield/web/modules/custom/tablefield/src/Element/Tablefield.php (485:3)
Drupal.Commenting.FunctionComment.ParamNameNoMatch at /builds/project/tablefield/web/modules/custom/tablefield/src/Element/Tablefield.php (488:6) -
erutan β
committed 0414748f on 8.x-2.x
Resolve #1677596 "Copy paste option"
-
erutan β
committed 0414748f on 8.x-2.x
- πΊπΈUnited States erutan
Params were copy-pasted from elsewhere in the document before.
- π¨π¦Canada Liam Morland Ontario, CA π¨π¦
Should this be cherry-picked onto branch 3.0.x?
- πΊπΈUnited States erutan
It's new functionality, but it's just an extra option on top of existing ones (and is very useful). It'd be nice to have it for 2.x so people with older versions of Drupal for whatever reason can use it IMO.
-
liam morland β
committed 7d727b49 on 3.0.x authored by
erutan β
Issue #1677596: Copy paste option
-
liam morland β
committed 7d727b49 on 3.0.x authored by
erutan β
-
liam morland β
committed 749ecbd0 on 3.0.x
Issue #1677596: Correct spelling error Fixes issue introduced in...
-
liam morland β
committed 749ecbd0 on 3.0.x
-
liam morland β
committed 9ad77efb on 8.x-2.x
Issue #1677596: Correct spelling error Fixes issue introduced in...
-
liam morland β
committed 9ad77efb on 8.x-2.x
Automatically closed - issue fixed for 2 weeks with no activity.