Skip to content
This repository has been archived by the owner on Jul 10, 2023. It is now read-only.

fix: don't apply region taxes to custom gift cards #561

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

dwene
Copy link

@dwene dwene commented May 11, 2022

When I'm in the admin UI and want to create a custom gift card, it shouldn't remove balance from the number I just input. I would understand charging taxes in a purchase, but if a store owner wants to give a gift card to someone with a round number ,they shouldn't have to finagle with the admin UI to get a nice round number.

https://www.loom.com/share/b81cee853ffb48f88d8792bc87d81aeb

Also fixing an issue where the "Balance Remaining" was showing the original amount in the table.

@netlify
Copy link

netlify bot commented May 11, 2022

Deploy Preview for nostalgic-booth-1cf149 ready!

Name Link
🔨 Latest commit 660e3c5
🔍 Latest deploy log https://app.netlify.com/sites/nostalgic-booth-1cf149/deploys/627c329753ccea000a088976
😎 Deploy Preview https://deploy-preview-561--nostalgic-booth-1cf149.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 site settings.

@netlify
Copy link

netlify bot commented May 11, 2022

Deploy Preview for palmes-staging-admin ready!

Name Link
🔨 Latest commit 660e3c5
🔍 Latest deploy log https://app.netlify.com/sites/palmes-staging-admin/deploys/627c32977af67d00080cb35b
😎 Deploy Preview https://deploy-preview-561--palmes-staging-admin.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 site settings.

@netlify
Copy link

netlify bot commented May 11, 2022

Deploy Preview for tekla-admin ready!

Name Link
🔨 Latest commit 660e3c5
🔍 Latest deploy log https://app.netlify.com/sites/tekla-admin/deploys/627c32977af67d00080cb35d
😎 Deploy Preview https://deploy-preview-561--tekla-admin.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 site settings.

@netlify
Copy link

netlify bot commented May 11, 2022

Deploy Preview for elastic-keller-bcaedb ready!

Name Link
🔨 Latest commit 660e3c5
🔍 Latest deploy log https://app.netlify.com/sites/elastic-keller-bcaedb/deploys/627c3297e822700008d55469
😎 Deploy Preview https://deploy-preview-561--elastic-keller-bcaedb.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 site settings.

@netlify
Copy link

netlify bot commented May 11, 2022

Deploy Preview for preeminent-gingersnap-0af4e1 ready!

Name Link
🔨 Latest commit 660e3c5
🔍 Latest deploy log https://app.netlify.com/sites/preeminent-gingersnap-0af4e1/deploys/627c3297e822700008d5546e
😎 Deploy Preview https://deploy-preview-561--preeminent-gingersnap-0af4e1.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 site settings.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant