Skip to content

IBX-11657: Added connector-quable to CI#1781

Open
katarzynazawada wants to merge 3 commits into5.0from
ibx-11657
Open

IBX-11657: Added connector-quable to CI#1781
katarzynazawada wants to merge 3 commits into5.0from
ibx-11657

Conversation

@katarzynazawada
Copy link
Copy Markdown
Contributor

@katarzynazawada katarzynazawada commented Apr 23, 2026

🎫 Issue IBX-11657

Related PRs:

Description:

This PR adds a new job for commerce regression with installed ibexa/connector quable. We cannot add this repository in the standard way, as connector-quable changes the appearance and functionality of the product catalog, causing tests related to products in any way to fail - that is why it has been added to a separate job.
I am adding this to the regression tests rather than to the repository package itself, so that the check is run more frequently, not just on PRs for connector-quable. An additional flag has been added to check whether connector-quable should be installed. At this stage, the main objective is to verify that the package installs correctly within the project. I have added the execution of admin-ui tests for basic coverage. Once dedicated tests for the connector-quable repository have been created, they could be run here.

For now, admin-ui test fails due to https://ibexa.atlassian.net/browse/IBX-11671. I think it’s worth waiting with the merge until the bug has been fixed.

For QA:

Documentation:

Copy link
Copy Markdown
Contributor

@micszo micszo left a comment

Choose a reason for hiding this comment

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

Thanks.

This is an optimal solution in the current situation.

Comment thread .github/workflows/browser-tests.yml Outdated
Comment thread .github/workflows/browser-tests.yml Outdated
@ibexa-yuna ibexa-yuna deleted the branch 5.0 April 29, 2026 08:11
@ibexa-yuna ibexa-yuna closed this Apr 29, 2026
@mnocon mnocon reopened this Apr 29, 2026
@mnocon mnocon changed the base branch from master to 5.0 April 29, 2026 09:09
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.

7 participants