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

U4X-267 : The function to delete or remove packaging unit entries is not working #45

Conversation

jabahum
Copy link
Collaborator

@jabahum jabahum commented Dec 13, 2023

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

Screenshots

Related Issue

https://metsprogramme.atlassian.net/browse/U4X-267

Other

@jabahum jabahum added the enhancement New feature or request label Dec 14, 2023
@jabahum jabahum self-assigned this Dec 14, 2023
@jabahum jabahum linked an issue Dec 14, 2023 that may be closed by this pull request
11 tasks
@jabahum jabahum marked this pull request as ready for review December 14, 2023 12:35
@slubwama slubwama changed the title feat(stock) :U4X-267 The function to delete or remove packaging unit entries is not working U4X-267 :The function to delete or remove packaging unit entries is not working Dec 14, 2023
@jabahum jabahum changed the title U4X-267 :The function to delete or remove packaging unit entries is not working U4X-267 : The function to delete or remove packaging unit entries is not working Dec 14, 2023
@jabahum jabahum removed a link to an issue Dec 14, 2023
11 tasks
@makombe
Copy link
Contributor

makombe commented Dec 18, 2023

@jabahum Please resolve the conflicts. Thanks

Comment on lines 121 to 123
title: `Delete packing item `,
kind: "success",
description: `Stock Item packing unit deleted Successfully`,
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add translations

Comment on lines 128 to 185
title: `Error Deleting a stock item packing unit`,
kind: "error",
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here translation of texts will be nice to have. This will come in handy for other implementers

…entries is not working

## Requirements
- [ ] This PR has a title that briefly describes the work done including a [conventional commit](https://o3-dev.docs.openmrs.org/#/getting_started/contributing?id=your-pr-title-should-indicate-the-type-of-change-it-is) type prefix and a Jira ticket number if applicable. See existing PR titles for inspiration.

#### For changes to apps
- [ ]  My work conforms to the [**OpenMRS 3.0 Styleguide**](https://om.rs/styleguide) and [**design documentation**](https://zeroheight.com/23a080e38/p/880723-introduction).

#### If applicable
- [ ] My work includes tests or is validated by existing tests.

## Summary
<!-- Please describe what problems your PR addresses. -->
<!-- *None* -->

## Screenshots
<!-- Required if you are making UI changes. -->
<!-- *None* -->

## Related Issue
<!-- Paste the link to the Jira ticket here if one exists. -->
<!-- https://issues.openmrs.org/browse/O3- -->
<!-- *None* -->

## Other
<!-- Anything not covered above -->
<!-- *None* -->
@jabahum jabahum force-pushed the ft/The-function-to-delete-or-remove-packaging-unit-entries-is-not-working branch from 1007e3e to ab76d00 Compare December 19, 2023 12:49
@Mwanje Mwanje merged commit 1df1e47 into openmrs:main Dec 20, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants