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) : Add ability to capture reference code for defined mode of payments #526

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

donaldkibet
Copy link
Contributor

@donaldkibet donaldkibet commented Dec 23, 2024

Requirements

  • This PR has a title that briefly describes the work done, including the ticket number if there is a ticket.
  • My work conforms to the OpenMRS 3.0 Styleguide.
  • I checked for feature overlap with existing widgets.

Summary

This PR adds ability to capture reference code for mode of payments that have the correct attribute set e.g MPESA Reference code for mobile money mode of payment. In addition to this, i have added conditionally display of the reference code field to ensure its applied when need.

Screenshots

Kapture.2025-01-07.at.14.17.19.webm

Related Issue

None.

Other

None.

@donaldkibet donaldkibet force-pushed the feat/abilityToCaptureReference branch from 35f3c35 to c8cae68 Compare January 7, 2025 11:40
@donaldkibet donaldkibet marked this pull request as ready for review January 7, 2025 11:40
@donaldkibet donaldkibet changed the title Feat/ability to capture reference (feat) : Add ability to capture reference code for defined mode of payments Jan 7, 2025
Copy link
Collaborator

@makombe makombe left a comment

Choose a reason for hiding this comment

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

Thanks @donaldkibet

@makombe makombe merged commit ad56e2d into main Jan 7, 2025
4 checks passed
@makombe makombe deleted the feat/abilityToCaptureReference branch January 7, 2025 12:37
@ojwanganto
Copy link
Contributor

Thanks, @donaldkibet. I have observed a UUID like string in the service name. Where is that coming from?

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