Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added purchase_order_number to transactions.json #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dhenry-cravedelivery
Copy link

Description of change

type: null/string
Description from api reference:
"purchaseOrderNumber: String
A purchase order identification value you associate with this transaction."

Manual QA steps

  • tested locally

Risks

Rollback steps

  • revert this branch

@TheBigDaddy42
Copy link

@luandy64 @nick-mccoy Wanted to follow up on the request above and see if you have any feedback @dhenry-cravedelivery

@TheBigDaddy42
Copy link

@luandy64 @nick-mccoy Hey Guys, wanted to follow up on this again. Are you able to look into this for @dhenry-cravedelivery ? Thank you!

@dhenry-cravedelivery
Copy link
Author

We need this data field to link our orders with braintree transactions; is there anything further we need to do on our end so this update can be merged?

@TheBigDaddy42
Copy link

TheBigDaddy42 commented Oct 13, 2021

This is beginning to impact our work. Can you please look into this, and let us know if there is anything needed on our end?

@TheBigDaddy42
Copy link

@luandy64 @nick-mccoy Followed up with you both a few times now, and am really looking for your help on this item as it is starting to impact our work. Can you please look into this and provide any feedback?

@TheBigDaddy42
Copy link

@luandy64 @nick-mccoy Hey Guys - wanted to follow up again here. Is there anything that I can do on my end to get this process moving here?

@TheBigDaddy42
Copy link

@luandy64 @nick-mccoy Following up again on this one. Are either of you able to provide any feedback?

@TheBigDaddy42
Copy link

TheBigDaddy42 commented Nov 8, 2021

@luandy64 @nick-mccoy Following up here. Did you have any feedback on this? Please let me know if there is anything other users that I should be reaching out to.

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