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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PLA-309] fix: don't set translation if no element #2454

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

P1X3L
Copy link
Contributor

@P1X3L P1X3L commented Jun 6, 2024

An error in sentry has shown that style could be called on undefined here 馃憠 https://github.com/WTTJ/welcome-ui/blob/main/packages/Breadcrumb/src/index.tsx#L66

(the sentry error for those who have access 馃憠 https://wttj.sentry.io/issues/5458739338/?environment=production&project=1222380&query=is:unresolved+issue.priority:%5Bhigh,+medium%5D&statsPeriod=1h&stream_index=0)

This is just about fixing the error when no element.

Signed-off-by: Paul-Xavier Ceccaldi <[email protected]>
@P1X3L P1X3L requested a review from a team as a code owner June 6, 2024 19:51
Copy link

github-actions bot commented Jun 6, 2024

馃憖 Visit Preview

@P1X3L P1X3L merged commit 968522c into main Jun 7, 2024
13 checks passed
@P1X3L P1X3L deleted the fix-error-on-breadcrumb branch June 7, 2024 18:32
@P1X3L P1X3L changed the title fix: don't set translation if no element [PLA-309] fix: don't set translation if no element Jun 13, 2024
Copy link

linear bot commented Jun 13, 2024

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

Successfully merging this pull request may close these issues.

None yet

2 participants