- Issue created by @arunkumark
- @arunkumark opened merge request.
- Status changed to Needs review
over 1 year ago 8:49am 16 February 2023 - 🇨🇦Canada gregcube
I like this idea, but it needs some work still.
$route_params = [ 'entity_type' => 'commerce_order', 'entity_id' => 1, 'export_type' => 'pdf' ];
We can't always assume entity id 1.
We should also add a requirement for entity_print or wrap something like:
if (\Drupal::moduleHandler()->moduleExists('entity_print')) { // Code Here }
around it.
I think it would also be better implemented as a local action item.
- Status changed to Needs work
over 1 year ago 10:27pm 27 February 2023 - 🇮🇳India arunkumark Coimbatore
@gregcube
Adding the Local action item is better than having the Print link. - Status changed to Needs review
over 1 year ago 11:33pm 28 February 2023 - 🇨🇦Canada gregcube
Give the latest a try. Let me know how it works for you.
- 🇮🇳India arunkumark Coimbatore
@gregcube
The updates look awesome. Attached screenshot of the testing. Good to move for RTBC, but keep any Needs Review to check by someone else.
++RTBC
- 🇺🇸United States RyanCMcConnell
Any idea what the holdup on this is? I'm planning on using the issue fork for my current project but I'd love if this was just the standard feature.
- Status changed to Fixed
about 1 year ago 10:45pm 30 October 2023 Automatically closed - issue fixed for 2 weeks with no activity.