Skip to content

OBPIH-6320 show supplier product name in tooltips#4929

Merged
awalkowiak merged 7 commits intodevelopfrom
feature/OBPIH-6320-show-supplier-product-name-in-tooltips
Nov 8, 2024
Merged

OBPIH-6320 show supplier product name in tooltips#4929
awalkowiak merged 7 commits intodevelopfrom
feature/OBPIH-6320-show-supplier-product-name-in-tooltips

Conversation

@drodzewicz
Copy link
Collaborator

@drodzewicz drodzewicz commented Nov 7, 2024

✨ Description of Change

A concise summary of what is being changed. Please provide enough context for reviewers to be able to understand the change and why it is necessary. If the issue/ticket already provides enough information, you can put "See ticket" as the description.

Link to GitHub issue or Jira ticket: OBPIH-6320

Description:


📷 Screenshots & Recordings (optional)

If this PR contains a UI change, consider adding one or more screenshots here or link to a screen recording to help reviewers visualize the change. Otherwise, you can remove this section.

@drodzewicz drodzewicz self-assigned this Nov 7, 2024
@github-actions github-actions bot added type: feature A new piece of functionality for the app domain: frontend Changes or discussions relating to the frontend UI domain: backend Changes or discussions relating to the backend server labels Nov 7, 2024
</td>
</g:if>
<td style="color: ${orderItem?.product?.color}">
<td style="color: ${orderItem?.product?.color}" title="hello">
Copy link
Collaborator

Choose a reason for hiding this comment

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

hello 👋

Comment on lines 89 to 91
const productSupplierName = orderItem?.productSupplierName
? `(source: ${orderItem?.productSupplierName})`
: null;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe it can be moved to a separate function for getting productSupplierName? I can see that it is used more than once

@awalkowiak awalkowiak merged commit e0a168d into develop Nov 8, 2024
@awalkowiak awalkowiak deleted the feature/OBPIH-6320-show-supplier-product-name-in-tooltips branch November 8, 2024 13:04
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 type: feature A new piece of functionality for the app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants