-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[AC 480]Add the organization name to the stripe invoice #2772
[AC 480]Add the organization name to the stripe invoice #2772
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! I don't know how the invoice layout works in the Stripe dashboard, so I'm assuming that you've checked that out and this is the metadata we need.
Just 1 very minor formatting nit. You can fix it and then move the ticket to QA in the meantime, I'll approve tomorrow.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good but I left a suggestion regarding the custom fields that I think would make the code easier to maintain.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
If Rui's happy then I'm happy ;) |
Type of change
Objective
Add Organization Name to the stripe invoice
Code changes
Before you submit
dotnet format --verify-no-changes
) (required)