Skip to content

[Behat] IBX-9747: Added scenario for creating URL Alias #1606

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 22 commits into from
Oct 9, 2025
Merged

[Behat] IBX-9747: Added scenario for creating URL Alias #1606

merged 22 commits into from
Oct 9, 2025

Conversation

barbaragr
Copy link
Contributor

@barbaragr barbaragr commented Jun 23, 2025 •
edited
Loading

🎫 Issue IBX-9747

Related PRs:

Description:

Added scenarios for direct and redirect url aliases:
Screenshot 2025-10-06 at 15 17 54

For QA:

OSS tests fail because of dependencies to Product Catalog. They were successful before adding dependencies: https://github.com/ibexa/admin-ui/actions/runs/18225264713/job/51894741617

Documentation:

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
4.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@barbaragr barbaragr force-pushed the ibx-9747-add-behat-tests-coverage-for-URL-aliases branch from 9de14fd to 2006057 Compare September 3, 2025 13:15
@barbaragr barbaragr marked this pull request as ready for review October 6, 2025 05:08
@barbaragr barbaragr mentioned this pull request Oct 6, 2025
Closed
@barbaragr barbaragr requested a review from a team October 6, 2025 13:55
@ezrobot ezrobot requested review from micszo and removed request for a team October 6, 2025 13:55
KamilSznajdrowicz
KamilSznajdrowicz requested changes Oct 6, 2025
View reviewed changes
Copy link
Contributor

@KamilSznajdrowicz KamilSznajdrowicz left a comment

Choose a reason for hiding this comment

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

Nice 🚀
Just a few nitpicks :)

  • please add [Behat] to your PR title :)

@barbaragr barbaragr changed the title IBX-9747: Added scenario for creating URL Alias IBX-9747: [Behat] Added scenario for creating URL Alias Oct 7, 2025
@barbaragr barbaragr changed the title IBX-9747: [Behat] Added scenario for creating URL Alias [Behat] IBX-9747: Added scenario for creating URL Alias Oct 7, 2025
tomaszszopinski
tomaszszopinski reviewed Oct 7, 2025
View reviewed changes
tomaszszopinski
tomaszszopinski reviewed Oct 8, 2025
View reviewed changes
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Oct 8, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
3.8% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@katarzynazawada katarzynazawada merged commit e08c1fb into 4.6 Oct 9, 2025
22 of 28 checks passed
@katarzynazawada katarzynazawada deleted the ibx-9747-add-behat-tests-coverage-for-URL-aliases branch October 9, 2025 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomaszszopinski tomaszszopinski tomaszszopinski approved these changes

@katarzynazawada katarzynazawada katarzynazawada approved these changes

@KamilSznajdrowicz KamilSznajdrowicz KamilSznajdrowicz approved these changes

@micszo micszo Awaiting requested review from micszo

@pawlakadrian pawlakadrian Awaiting requested review from pawlakadrian

@juskora juskora Awaiting requested review from juskora

Assignees

@mikadamczyk mikadamczyk

@ciastektk ciastektk

@dew326 dew326

@ViniTou ViniTou

Labels
Ready for review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@barbaragr @katarzynazawada @tomaszszopinski @KamilSznajdrowicz @mikadamczyk @ciastektk @dew326 @ViniTou