Skip to content

IBX-11567: Add translations management to CI Script#136

Merged
micszo merged 1 commit into
mainfrom
ibx-xxxx-add-translations-management-to-ci-script
Jul 23, 2026
Merged

IBX-11567: Add translations management to CI Script#136
micszo merged 1 commit into
mainfrom
ibx-xxxx-add-translations-management-to-ci-script

Conversation

@KamilSznajdrowicz

@KamilSznajdrowicz KamilSznajdrowicz commented May 18, 2026

Copy link
Copy Markdown
Contributor
🎫 Issue IBX-11567

Related PRs:

Description:

The translations-managment feature is only available starting from the Headless version.

For QA:

Regression: ibexa/commerce#1854
ibexa/oss#282

Documentation:

@sonarqubecloud

Copy link
Copy Markdown

@micszo micszo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assuming it has been confirmed the package will stay opt-in on v6.

@KamilSznajdrowicz

KamilSznajdrowicz commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

Assuming it has been confirmed the package will stay opt-in on v6.

Good catch 👍
Since translations-management will become part of the product core in v6, I see two options: either I add it now and remove it later once it becomes part of the core product, or I remove it from v6.0 now and wait until it is officially included in the product.
The main concern is that Bulk Translation, which is part of translations-management will be released later, is only available in v6. Because of that, it seems to me that the best option for now is to remove the configuration from v6 and merge and test only v5. What do you think? @micszo

@micszo

micszo commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Thanks for confirming.

Let's go for option one.

@KamilSznajdrowicz
KamilSznajdrowicz force-pushed the ibx-xxxx-add-translations-management-to-ci-script branch from c9c3ad5 to 74fbcd8 Compare July 22, 2026 21:06
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

💡 Need a hand with PR review? Try Gitar by Sonar!

@micszo
micszo merged commit 8f99d8f into main Jul 23, 2026
5 of 6 checks passed
@micszo
micszo deleted the ibx-xxxx-add-translations-management-to-ci-script branch July 23, 2026 06:49
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.

3 participants