Skip to content

Include receiver name in DeliveryHeader - #213

Merged
jeffersontuc merged 1 commit into
vtex-apps:masterfrom
jay-costa:patch-1
Apr 16, 2026
Merged

jeffersontuc merged 1 commit into
vtex-apps:masterfrom
jay-costa:patch-1

Conversation

@jay-costa

@jay-costa jay-costa commented Apr 15, 2026 •

Copy link
Copy Markdown
Contributor

Add receiver name display to DeliveryHeader component. Change requested through ticket #1393065.

What is the purpose of this pull request?

Add the receiverName to the orders placed page when the order has delivery as the shipping method.

How should this be manually tested?

Use this workspace https://testorder--desafioonboarding30.myvtex.com/checkout/cart/add/?sku=92&qty=1&seller=1&sc=1

Screenshots or example usage

current behavior
image

Expected Behavior
image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires change to documentation, which will be updated after the PR's approval.

Add receiver name display to DeliveryHeader component. Change requested through ticket #1393065.
@jay-costa
jay-costa requested a review from a team as a code owner April 15, 2026 18:21
@jay-costa
jay-costa requested review from guilhermedias-bit and vmarcosp and removed request for a team April 15, 2026 18:21
@vtex-io-ci-cd

vtex-io-ci-cd Bot commented Apr 15, 2026 •

Copy link
Copy Markdown
Contributor

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

  • Patch (backwards-compatible bug fixes)

  • Minor (backwards-compatible functionality)

  • Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

  • No thanks, I would rather do it manually 😞

@vtex-io-docs-bot

vtex-io-docs-bot Bot commented Apr 15, 2026 •

Copy link
Copy Markdown

Beep boop 🤖

I noticed you didn't make any changes at the docs/ folder

  • There's nothing new to document 🤔
  • I'll do it later 😞

In order to keep track, I'll create an issue if you decide now is not a good time

  • I just updated 🎉🎉

@jeffersontuc
jeffersontuc merged commit 0c9a28a into vtex-apps:master Apr 16, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants