Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
Currently the "shipments" field is not present in the field names whitelist.
As a consequence, its relationship/field is not returned alongside the order.
Comment | File | Size | Author |
---|---|---|---|
#9 | 3162425-do-not-test.patch | 633 bytes | jsacksick |
#8 | 3162425-8.patch | 9.1 KB | jsacksick |
| |||
#2 | Add_shipments_field_3162425.patch | 542 bytes | Pooja Ganjage |
Comments
Comment #2
Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commentedHi,
I am applying patch for this issue.
Kindly review this patch once.
Thanks.
Comment #3
Pooja Ganjage CreditAttribution: Pooja Ganjage at Asentech LLC commentedComment #4
mglamanThanks @ Pooja Ganjage!
The patch was generated incorrectly. It looks like the patch was generated from a different repository. Next time, make sure you are in the proper directly and it will apply :)
Comment #6
mglaman🥳 Committed!
Comment #7
jsacksick CreditAttribution: jsacksick at Centarro commentedThe tests had to be updated as well, they're currently failing because this changes the payload...
Comment #8
jsacksick CreditAttribution: jsacksick at Centarro commentedI'm wondering if this require Commerce shipping to be patched though, in which case, we need to commit the Commerce shipping patch from #3174306: Viewing shipments requires the "administer commerce_shipment" permission and require Commerce shipping dev until the next release is tagged.
Comment #9
jsacksick CreditAttribution: jsacksick at Centarro commentedAttaching the "shipments" patch that applies.
Comment #11
jsacksick CreditAttribution: jsacksick at Centarro commentedCommitting since this fixes the broken tests.