Skip to content

OBPIH-6397 Full outbound import - fix QA issues#4787

Merged
awalkowiak merged 1 commit intofeature/outbound-importfrom
OBPIH-6397-qa-fix
Aug 14, 2024
Merged

OBPIH-6397 Full outbound import - fix QA issues#4787
awalkowiak merged 1 commit intofeature/outbound-importfrom
OBPIH-6397-qa-fix

Conversation

@kchelstowski
Copy link
Collaborator

QA issues fixed:

  • make the recipient nullable and change the error message if the recipient was provided, but not found
  • persist tracking number to make it visible on SM view page
  • unify ship date on create full outbound page and on outbound send page (time context)
  • validate the empty packing list (do not allow to import an empty template)
  • validate quantity picked 0.
  • make the expiration date not bindable, and display it in the table after associating the inventory item with the provided lot number
  • improve display of validation errors on frontend (a case when we are on confirm page and for example recall a lot on second tab and then run finish)
  • unify the name of date shipped field
  • fix issue with "empty" transaction entries being persisted, that then caused the transaction view page not working

@kchelstowski kchelstowski self-assigned this Aug 14, 2024
@github-actions github-actions bot added domain: frontend Changes or discussions relating to the frontend UI domain: backend Changes or discussions relating to the backend server domain: l10n Changes or discussions relating to localization & Internationalization labels Aug 14, 2024
@awalkowiak awalkowiak merged commit 30483c3 into feature/outbound-import Aug 14, 2024
@awalkowiak awalkowiak deleted the OBPIH-6397-qa-fix branch August 14, 2024 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: backend Changes or discussions relating to the backend server domain: frontend Changes or discussions relating to the frontend UI domain: l10n Changes or discussions relating to localization & Internationalization

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants