Skip to content

Conversation

@max-koro
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Nov 27, 2025

Deploy Preview for incomparable-tiramisu-91a96a ready!

Name Link
🔨 Latest commit a055426
🔍 Latest deploy log https://app.netlify.com/projects/incomparable-tiramisu-91a96a/deploys/692ee1088697d000082ad1b1
😎 Deploy Preview https://deploy-preview-496--incomparable-tiramisu-91a96a.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

<details>
<summary>Query original delivery address</summary>
```graphql
query PhysicalCardAddress {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why, but this query doesn't work for me. Does it work for you? What would I need to do to make it work?

It may be an issue on my side. I get this as a result:

{
  "data": {
    "card": {
      "physicalCard": {
        "statusInfo": {}
      }
    }
  }
}

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's probably an issue on my side. Can you please double-check?

Copy link
Contributor Author

@max-koro max-koro Dec 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The query is correct, the issue is that your physical card isn't in ToRenew status.

The inline fragment ... on PhysicalCardToRenewStatusInfo only returns data when the card matches that specific status.

To test this, you need a physical card in ToRenew status.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried it with two cards in ToRenew status. So it may just be an issue with my execution. You've tested it, and it works, so it's good to go.

Copy link
Contributor

@ntombing ntombing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, this is good! I left a suggestion and a comment about the query.

Can you please double-check it? :)

Co-authored-by: ntombing <mantombingoloyi@gmail.com>
@max-koro max-koro merged commit 23ec60d into main Dec 2, 2025
4 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.

3 participants