Skip to content

OBPIH-6328 Dashboard indicator for items with backdated shipments (disable string truncation)#4649

Merged
awalkowiak merged 4 commits intodevelopfrom
OBPIH-6328-3
Jun 5, 2024
Merged

OBPIH-6328 Dashboard indicator for items with backdated shipments (disable string truncation)#4649
awalkowiak merged 4 commits intodevelopfrom
OBPIH-6328-3

Conversation

@alannadolny
Copy link
Collaborator

@alannadolny alannadolny commented Jun 4, 2024

Things added after Manon's request:

  • colours: green (1 shipment) > blue > orange > red (4+)
  • instead of "4 or more shipments", just "4 or more"
  • hyperlink the shipment IDs and the product codes
  • date format: should be 31-Aug-2021
  • narrowing columns

@alannadolny alannadolny added the warn: do not merge Marks a pull request that is not yet ready to be merged label Jun 4, 2024
@alannadolny alannadolny removed the warn: do not merge Marks a pull request that is not yet ready to be merged label Jun 4, 2024
ColorNumber twoDelayedShipments = new ColorNumber(value: amountOfItemsWithBackdatedShipments[Constants.TWO], subtitle: '2 shipments', order: 2)
ColorNumber threeDelayedShipments = new ColorNumber(value: amountOfItemsWithBackdatedShipments[Constants.THREE], subtitle: '3 shipments', order: 3)
ColorNumber fourOrMoreDelayedShipments = new ColorNumber(value: amountOfItemsWithBackdatedShipments[Constants.FOUR_OR_MORE], subtitle: '4 or more shipments', order: 4)
ColorNumber fourOrMoreDelayedShipments = new ColorNumber(value: amountOfItemsWithBackdatedShipments[Constants.FOUR_OR_MORE], subtitle: '4 or more', order: 4)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it is worth translating these labels now.

DATE_FORMAT(stock_count.last_stock_count, "%d-%M-%Y") FROM (
(
SELECT t.id as transaction_id, s.shipment_number FROM shipment s
DATE_FORMAT(stock_count.last_stock_count, "%d-%b-%Y"),
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am assuming that we are leaving this as is for now, but I think this will have to be covered as a part of localized date tickets in the near future.

@alannadolny alannadolny requested a review from awalkowiak June 5, 2024 09:12
@awalkowiak awalkowiak merged commit 61dab31 into develop Jun 5, 2024
@awalkowiak awalkowiak deleted the OBPIH-6328-3 branch June 5, 2024 11:59
drodzewicz pushed a commit that referenced this pull request Jun 11, 2024
…sable string truncation) (#4649)

* OBPIH-6328 Add disabling truncation

* OBPIH-6328 change colouring, fix labels, add hyperlinks to products and shipments

* OBPIH-6328 Fix columns length

* OBPIH-6328 Add labels for subtitles and headers
drodzewicz pushed a commit that referenced this pull request Jun 11, 2024
…sable string truncation) (#4649)

* OBPIH-6328 Add disabling truncation

* OBPIH-6328 change colouring, fix labels, add hyperlinks to products and shipments

* OBPIH-6328 Fix columns length

* OBPIH-6328 Add labels for subtitles and headers
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