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

Add Swedish translations #2445

Merged
merged 6 commits into from
Apr 16, 2024
Merged

Add Swedish translations #2445

merged 6 commits into from
Apr 16, 2024

Conversation

atg-pablo-widlert
Copy link
Contributor

Context

Add Swedish translations for Allure

Checklist

@atg-pablo-widlert
Copy link
Contributor Author

atg-pablo-widlert commented Mar 25, 2024

I guess this needs another label type:improvement to continue with the automated checks.

@baev baev added the type:new feature Change that add something new for end users label Apr 1, 2024
@atg-pablo-widlert
Copy link
Contributor Author

@baev Can you review + merge?

Copy link
Member

@baev baev left a comment

Choose a reason for hiding this comment

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

Please, add translations for the plugins:

plugins/behaviors-plugin/src/dist/static/index.js
plugins/packages-plugin/src/dist/static/index.js

Then, you need to enable translation in

allure-generator/src/main/javascript/utils/translation.js

and allow it to be configured via command-line option:

allure-commandline/src/main/java/io/qameta/allure/validator/LanguageValidator.java

@atg-pablo-widlert
Copy link
Contributor Author

@baev Can you merge this too? I am not authorized.

@baev baev changed the title Add Swedish translations for Allure Add Swedish translations Apr 16, 2024
@baev baev merged commit 3fc70b8 into allure-framework:main Apr 16, 2024
9 checks passed
@baev
Copy link
Member

baev commented Apr 16, 2024

@atg-pablo-widlert thanks!

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.

2 participants