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

fix: table messages for no record #1284

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Usmanfee
Copy link
Contributor

@Usmanfee Usmanfee commented Oct 30, 2024

Description

Added Translation of messages for PageLoading component for both EN and DE.

Change made in this PR will be a breaking changes since the PageLoading component from the shared-component expect a new prop of mappedTableSearchTranslation which is already done in another PR of shared-component repo:
eclipse-tractusx/portal-shared-components#369

Changelog entry:

- **PageLoading Table**
  - Added Translation for messages in EN and DE [#1286](https://github.com/eclipse-tractusx/portal-frontend/issues/1286)

Why

PageLoading component displaying the hardcoded message for no data for to shown in Table component and for no data based on search text.

Issue

#1286

Checklist

  • I have performed a self-review of my own code
  • I have successfully tested my changes locally

Copy link

@Usmanfee
Copy link
Contributor Author

Please also review dependent PR from shared-component: eclipse-tractusx/portal-shared-components#369. Thanks

Copy link
Contributor

@ss-nikunj ss-nikunj left a comment

Choose a reason for hiding this comment

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

Hi @Usmanfee,
I would suggest that we should take such kind of modification in portal-shared-component.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants