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: error due to l10n exported with "default" field. #1271

Merged
merged 3 commits into from
Nov 2, 2021

Conversation

OhotnikovM
Copy link
Contributor

@OhotnikovM OhotnikovM commented Oct 29, 2021

Hello!
We have an error due to this part of code because it is outside the component and after build it causes error because this code use global l10n module in bundle with nested default filed. That is we should apply it like l10n.default.en and etc or replace this code into component module.
Our team would be eternally grateful if you approve this pull request!

Contributes to #

  • carbon-components-vue

## What did you do?

  • We have an error due to this part of code because it is outside the component and after build it causes error because this code use global l10n module in bundle with nested default filed. That is we should apply it like l10n.default.en and etc or replace this code into component module.

## Why did you do it?

  • replaced part of code inside component module from global scope

How have you tested it?

  • i did it locally and tested

Were docs updated if needed?

  • No
  • N/A
  • No
  • Yes

Hello!
We have an error due to this part of code because it is outside the component and after build it causes error because this code use global l10n module in bundle with nested `default` filed. That is we should apply it like `l10n.default.en` and etc or replace this code into component module.
Our team would be eternally grateful if you approve this pull request!
@OhotnikovM OhotnikovM changed the title Error due to l10n exports with default field. Fix: error due to l10n exports with default field. Oct 29, 2021
@OhotnikovM OhotnikovM changed the title Fix: error due to l10n exports with default field. Fix: error due to l10n exported with default field. Oct 29, 2021
@OhotnikovM OhotnikovM changed the title Fix: error due to l10n exported with default field. Fix: error due to l10n exported with "default" field. Oct 29, 2021
@netlify
Copy link

netlify bot commented Oct 29, 2021

✔️ Deploy Preview for carbon-components-vue ready!

🔨 Explore the source changes: e648a88

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-components-vue/deploys/6181758651559c0008f51e96

😎 Browse the preview: https://deploy-preview-1271--carbon-components-vue.netlify.app

@lee-chase lee-chase changed the title Fix: error due to l10n exported with "default" field. fix: error due to l10n exported with "default" field. Nov 2, 2021
@OhotnikovM
Copy link
Contributor Author

@lee-chase Hello! It seems that i can't merge it myself due to protected branch. Could you merge it please?

@lee-chase
Copy link
Member

Will do. Just waiting for the ci/build to finish. Don't know why ci/circleci is still there.

@OhotnikovM
Copy link
Contributor Author

I got it. Thank you so much!

@lee-chase lee-chase merged commit 97781a4 into carbon-design-system:main Nov 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants