Skip to content

[Behat] IBX-10592: Implemented basic behat coverage for notification feature#25

Merged
micszo merged 2 commits into4.6from
ibx-10592-cover-notifications-feature
Dec 12, 2025
Merged

[Behat] IBX-10592: Implemented basic behat coverage for notification feature#25
micszo merged 2 commits into4.6from
ibx-10592-cover-notifications-feature

Conversation

@KamilSznajdrowicz
Copy link
Contributor

@KamilSznajdrowicz KamilSznajdrowicz commented Oct 27, 2025

@KamilSznajdrowicz KamilSznajdrowicz requested a review from a team October 27, 2025 13:18
@ezrobot ezrobot requested review from barbaragr, juskora, katarzynazawada, micszo, pawlakadrian and tomaszszopinski and removed request for a team October 27, 2025 13:18
name: "Notifications browser tests - Experience"
uses: ibexa/gh-workflows/.github/workflows/browser-tests.yml@main
with:
project-edition: 'experience'

Choose a reason for hiding this comment

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

Is experience the lowest edition we can test this on? ibexa/workflow has 'headless' also - is there any particular reason to only test exp here?

Copy link
Contributor Author

@KamilSznajdrowicz KamilSznajdrowicz Nov 18, 2025

Choose a reason for hiding this comment

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

In the first iteration I will release with commented headless mode due to the changed dashboard and have to test scenarios

Copy link

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

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

I am approving this config for experience. We've agreed with @KamilSznajdrowicz that headless setup will be a followup (due to differences in dashboard).

@micszo micszo merged commit c509dbf into 4.6 Dec 12, 2025
36 of 39 checks passed
@micszo micszo deleted the ibx-10592-cover-notifications-feature branch December 12, 2025 08:58
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

Comments