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

OP-1344: dialog method takes bundle key and not the final text #2060

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

dbmalkovsky
Copy link
Collaborator

Found while testing new jars

@dbmalkovsky dbmalkovsky requested a review from mwithi October 2, 2024 18:55
Copy link
Member

@mwithi mwithi left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks a lot!

@mwithi mwithi merged commit 8876f0f into informatici:develop Oct 2, 2024
1 check passed
mwithi added a commit that referenced this pull request Oct 4, 2024
* add validate action

* Update bundle/language_en.properties

* update logique and apply the new

* update  src/main/java/org/isf/medicalinventory/gui/InventoryEdit.java

* update bundle to add new message

* Rename one field

* Remove useless field and format code

* Remove deprecated field

* Fix exception after focus out on real qty

* Update one label

* update logic of validate Inventory

* update InventoryEdit

* update validate method

* update InventoryEdit

* update validate inventory'

* update Inventory Edit

* update InventoryEdit.java

* git fix error Index out of bound

* update Inventory

* apply suggestion

* fix error on update Cost of lot

* OP-1344: dialog method takes bundle key and not the final text (#2060)

* Add bundles

* Refactor validation method and change errors severity

* create new inventory during validation phase if we have new charge movement created after the creation date of inventory

---------

Co-authored-by: ArnaudFonzam <[email protected]>
Co-authored-by: David B Malkovsky <[email protected]>
Co-authored-by: Alessandro Domanico <[email protected]>
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.

2 participants