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

Update sv-SE.ts #4292

Merged
merged 6 commits into from
Oct 28, 2024
Merged

Update sv-SE.ts #4292

merged 6 commits into from
Oct 28, 2024

Conversation

MarcusT96
Copy link
Contributor

@MarcusT96 MarcusT96 commented Oct 6, 2024

Description

Edited the swedish localization package to more match the en-US package that is managed by the clerk team. This to make us swedes have a more complete authentication system in swedish and not mix swedish and english as before.

Checklist

  • npm test runs as expected.
  • npm run build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Edited the swedish localization package to more match the en-US package that is managed by the clerk team. This to make us swedes have a more complete authentication system in swedish and not mix swedish and english as before.
Copy link

changeset-bot bot commented Oct 6, 2024

🦋 Changeset detected

Latest commit: fb894ae

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

This PR includes changesets to release 4 packages
Name Type
@clerk/localizations Patch
@clerk/clerk-js Patch
@clerk/chrome-extension Patch
@clerk/clerk-expo Patch

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

Copy link
Member

@anagstef anagstef left a comment

Choose a reason for hiding this comment

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

Thank you for contributing! 🙏

packages/localizations/src/sv-SE.ts Outdated Show resolved Hide resolved
Copy link
Contributor Author

@MarcusT96 MarcusT96 left a comment

Choose a reason for hiding this comment

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

Left notes on the ones were I might have gone a bit bananas, sorry!

Copy link
Member

@tmilewski tmilewski left a comment

Choose a reason for hiding this comment

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

@MarcusT96 Maybe I'm misunderstanding some of your comments, but I think it makes sense to keep some of the original translations I called out here.

Thanks again!

packages/localizations/src/sv-SE.ts Outdated Show resolved Hide resolved
packages/localizations/src/sv-SE.ts Outdated Show resolved Hide resolved
@MarcusT96
Copy link
Contributor Author

Fixed the reverts, we should be good to go now!

@tmilewski tmilewski merged commit 159877b into clerk:main Oct 28, 2024
3 of 4 checks passed
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.

4 participants