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

feat(lpe): add discretionary reason #42135

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

Conversation

blaggacao
Copy link
Contributor

Context

  • LP may be earned on discretionary terms implemented in a servers script outside of the calculation of the loyalty program

  • Eg Referral bonus on third party sales

  • Communication / Notification may need to be distinguishable as per the reason

Solution

  • Mark mandatory fields as such
  • Add reason field to capture nuances for messaging

@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label Jul 1, 2024
@blaggacao blaggacao changed the title feat(lpe): add distretionary reason feat(lpe): add discretionary reason Jul 2, 2024
@blaggacao
Copy link
Contributor Author

Fears are nothing more than a state of mind. - Napoleon Hill


Kindly help move this PR forward. Many thanks from Yours Sincerely!

@blaggacao
Copy link
Contributor Author

What the caterpillar calls the end of the world, the master calls a butterfly. - Richard Bach


Kindly help move this PR forward. Many thanks from Yours Sincerely!

@blaggacao
Copy link
Contributor Author

Watch your thoughts, they become your words Watch your words, they become your actions Watch your actions, they become your habits Watch your habits, they become your character Watch your character, it becomes your destiny. -


Kindly help move this PR forward. Many thanks from Yours Sincerely!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-tests This PR needs automated unit-tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant