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

CIWEMB-569: Add Credit Note Tokens #133

Conversation

shahrukh-compuco
Copy link
Contributor

Overview

This pr adds credit note tokens while emailing the credit note.

Before

There were no credit note tokens to be used in subject or body of the email.

After

Screenshot from 2023-12-12 14-44-40

@shahrukh-compuco shahrukh-compuco force-pushed the CIWEMB-569-add-credit-note-tokens branch from 6fdf2d0 to fba9d05 Compare December 12, 2023 09:55
Copy link
Member

@omarabuhussein omarabuhussein left a comment

Choose a reason for hiding this comment

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

@shahrukh-compuco left a note, can you also squash the two commits in this PR into one, a commit like CIWEMB-569: Update code is vague and does not help or imply anything when you need to go back to look at the commit history later in the future, also check this document: https://compucorp.atlassian.net/wiki/spaces/SD/pages/1520173078/Git+Commits

financeextras.php Outdated Show resolved Hide resolved
@shahrukh-compuco shahrukh-compuco force-pushed the CIWEMB-569-add-credit-note-tokens branch 2 times, most recently from 1adeda1 to 8dc2d2a Compare December 12, 2023 11:44
@shahrukh-compuco shahrukh-compuco force-pushed the CIWEMB-569-add-credit-note-tokens branch from 8dc2d2a to fe2900e Compare December 12, 2023 11:46
@shahrukh-compuco shahrukh-compuco merged commit bcf6536 into credit-note-changes-workstream Dec 13, 2023
2 checks passed
@shahrukh-compuco shahrukh-compuco deleted the CIWEMB-569-add-credit-note-tokens branch December 13, 2023 06:13
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