bigcommerce / catalyst

Catalyst - for Composable Commerce
https://catalyst-demo.site
MIT License
117 stars 153 forks source link

Option to provide non translated strings in a fallback locale #1659

Closed thebigrick closed 1 week ago

thebigrick commented 1 week ago

What/Why?

If a new string is added, the translation must be available in any language. If not, the dotted path notation will be displayed in its place. I introduced the capability of using a fallback language for the missing strings.

Testing

changeset-bot[bot] commented 1 week ago

🦋 Changeset detected

Latest commit: cc35708708301ea6053707c2cd91e18302835475

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | -------------------------- | ----- | | @bigcommerce/catalyst-core | Minor |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

vercel[bot] commented 1 week ago

@thebigrick is attempting to deploy a commit to the BigCommerce Platform Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
catalyst-latest ✅ Ready (Inspect) Visit Preview Nov 19, 2024 7:57pm
deini commented 1 week ago

Looks good, can we add a changeset please?

thebigrick commented 1 week ago

Looks good, can we add a changeset please?

I just provided one. I opted for a minor change rather than a patch since I do not consider the previous behavior a bug. Is this correct?