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) initial setup of etl administration esm #496

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

its-kios09
Copy link
Collaborator

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

Screenshots

ETLAdministration

None.

Related Issue

None.

Other

None.

@ojwanganto
Copy link
Contributor

Thanks, @its-kios09. This will go a long way in saving us many clicks to the 1.x UI

@donaldkibet
Copy link
Contributor

donaldkibet commented Dec 2, 2024

Thank you @its-kios09 this is great addition. @ojwanganto, regarding admin functionality - I believe we will need to have features like account management and ETL admin access implemented. While it's been some time, I recall the community had plans to develop an Admin UI for version 3.x. We should review these existing community plans to ensure our admin UI vision aligns with the broader community intended direction.

@its-kios09
Copy link
Collaborator Author

@donaldkibet Thank you, I agree with you. However, I was considering whether we could proceed with implementing the feature while simultaneously reviewing the community's existing plans for the Admin UI in version 3.x. Later, we can align our implementation with the broader plan. What are your thoughts on this approach?

@donaldkibet
Copy link
Contributor

Mine was a recommendation so that we do not end up duplicating efforts or spend time doing whats already done.

@ojwanganto
Copy link
Contributor

ojwanganto commented Dec 2, 2024

Sorry for my late response. The ETL admin is actually a feature in KenyaEMR and so won't be sorted through the work to migrate the admin UI from either 1.x/2.x. We can have something that can call endpoints for recreating/refreshing ETLs

@its-kios09
Copy link
Collaborator Author

final results @makombe @donaldkibet

ETL

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