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.

Sign up for GitHub

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

Jump to bottom

O3-2933 Add patient flag for billing app features #20

Merged
merged 1 commit into from
Mar 11, 2024
Merged

O3-2933 Add patient flag for billing app features #20

merged 1 commit into from
Mar 11, 2024

Conversation

CynthiaKamau
Copy link
Contributor

@CynthiaKamau CynthiaKamau commented Mar 6, 2024 •
edited
Loading

Requirements

  • This PR has a title that briefly describes the work done including a conventional commit type prefix and a Jira ticket number if applicable. See existing PR titles for inspiration.

For changes to apps

If applicable

  • My work includes tests or is validated by existing tests.

Summary

  • This PR introduces a billing module feature flag which hides billing related components in the patient chart and homepage unless the feature flag is true

Screenshots

Screen.Recording.2024-03-06.at.14.09.10.mov

Related Issue

Other

@CynthiaKamau CynthiaKamau marked this pull request as draft March 6, 2024 11:21
@CynthiaKamau CynthiaKamau requested review from pirupius, denniskigen, ODORA0 and ebambo and removed request for pirupius March 6, 2024 11:22
@CynthiaKamau CynthiaKamau marked this pull request as ready for review March 6, 2024 12:31
eudson
eudson approved these changes Mar 11, 2024
View reviewed changes
Copy link

@eudson eudson left a comment

Choose a reason for hiding this comment

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

LGTM

@pirupius pirupius merged commit fa039f0 into main Mar 11, 2024
5 checks passed
@pirupius pirupius deleted the O3-2933 branch March 11, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pirupius pirupius pirupius approved these changes

@eudson eudson eudson approved these changes

@denniskigen denniskigen Awaiting requested review from denniskigen

@ODORA0 ODORA0 Awaiting requested review from ODORA0

@ebambo ebambo Awaiting requested review from ebambo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@CynthiaKamau @eudson @pirupius