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) consolidated palladium fork changes #56

Merged
merged 15 commits into from
Jan 14, 2024
Merged

(feat) consolidated palladium fork changes #56

merged 15 commits into from
Jan 14, 2024

Conversation

donaldkibet
Copy link
Member

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 brings changes done on palladium kenya fork to main repository for better collaboration. This PR has the following significant changes

  1. Add delete confirmation dialog for stock sources
  2. (feat) revalidate stock operation on succesfully submission
  3. Error on save and on submission made user friendly
  4. (fix) upgrade @openmrs/esm-framework
  5. Showing stock operation types based on user roles (

Related Issue

Other

@donaldkibet donaldkibet changed the title (feat) Consolidate palladium fork changes (feat) consolidated palladium fork changes Jan 12, 2024
Copy link
Contributor

@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.

This is in order @donaldkibet Thanks.

@makombe makombe merged commit 9e96211 into openmrs:main Jan 14, 2024
3 checks passed
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.

6 participants