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

(Fix) Added Facility Setup page in the Administration app #550

Merged
merged 14 commits into from
Jan 17, 2025
Merged

(Fix) Added Facility Setup page in the Administration app #550

merged 14 commits into from
Jan 17, 2025

Conversation

FelixKiprotich350
Copy link
Contributor

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

  • Facility setup page has been added to administration app. This page is porting facility related info from sysinfo app including
  • SHA status, facility name and operational status.
  • Keph level and MFL code has been added.
  • It also has a button to pull and synchronize data from HIE.

Screenshots

Screenshot from 2025-01-15 19-26-09
Screenshot from 2025-01-15 19-26-14

Related Issue

None.

Other

None.

@FelixKiprotich350
Copy link
Contributor Author

@ojwanganto @njorocs @donaldkibet kindly review this

@FelixKiprotich350
Copy link
Contributor Author

@ojwanganto have a look

donaldkibet
donaldkibet previously approved these changes Jan 17, 2025
View reviewed changes
@donaldkibet
Copy link
Contributor

Please fix the conflicts and merge it in cc @FelixKiprotich350

@FelixKiprotich350
Copy link
Contributor Author

@donaldkibet done

donaldkibet
donaldkibet previously approved these changes Jan 17, 2025
View reviewed changes
@donaldkibet
Copy link
Contributor

@FelixKiprotich350 merge once the CI is completed checks

FelixKiprotich350
FelixKiprotich350 commented Jan 17, 2025
View reviewed changes
Copy link
Contributor Author

@FelixKiprotich350 FelixKiprotich350 left a comment

Choose a reason for hiding this comment

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

@its-kios09 done

@FelixKiprotich350
Copy link
Contributor Author

@donaldkibet assist

@its-kios09
Copy link
Collaborator

@FelixKiprotich350 check why CI build is failing I merge

its-kios09
its-kios09 previously approved these changes Jan 17, 2025
View reviewed changes
@FelixKiprotich350
Copy link
Contributor Author

@its-kios09 thanks am checking that

@FelixKiprotich350
Copy link
Contributor Author

@donaldkibet i think this is ready now

@donaldkibet donaldkibet merged commit 7e88a60 into palladiumkenya:main Jan 17, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ojwanganto ojwanganto ojwanganto left review comments

@its-kios09 its-kios09 its-kios09 left review comments

@donaldkibet donaldkibet donaldkibet left review comments

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

Successfully merging this pull request may close these issues.
4 participants
@FelixKiprotich350 @donaldkibet @its-kios09 @ojwanganto