Skip to content

IBX-10170: Refactored access check to use new contract interface #1603

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 3 commits into from
Jul 10, 2025
Merged

IBX-10170: Refactored access check to use new contract interface #1603

merged 3 commits into from
Jul 10, 2025

Conversation

@ViniTou
Copy link
Contributor

@ViniTou ViniTou commented Jun 18, 2025

🎫 Issue IBX-10170

Related PRs:

Description:

For QA:

Documentation:

@ViniTou ViniTou changed the title IBX-10170: Refactored access check to use new contract interface [TMP] IBX-10170: Refactored access check to use new contract interface Jun 18, 2025
@ViniTou ViniTou force-pushed the ibx-10170-access-check branch from 60b7368 to 58ef7d6 Compare June 18, 2025 13:04
@ViniTou ViniTou force-pushed the ibx-10170-access-check branch from 58ef7d6 to 430d996 Compare June 18, 2025 13:11
@ViniTou ViniTou requested a review from a team June 18, 2025 13:52
@ezrobot ezrobot requested review from Steveb-p, adamwojs, alongosz, barw4, ciastektk, mikadamczyk and wiewiurdp and removed request for a team June 18, 2025 13:52
@ViniTou ViniTou force-pushed the ibx-10170-access-check branch from 5bf2d6a to e060810 Compare July 9, 2025 12:19
@ViniTou ViniTou changed the title [TMP] IBX-10170: Refactored access check to use new contract interface IBX-10170: Refactored access check to use new contract interface Jul 9, 2025
@juskora juskora force-pushed the ibx-10170-access-check branch from e060810 to e259375 Compare July 10, 2025 10:29
@juskora juskora mentioned this pull request Jul 10, 2025
Closed
@juskora juskora force-pushed the ibx-10170-access-check branch from e259375 to 18b8e48 Compare July 10, 2025 11:56
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@juskora
Copy link
Contributor

juskora commented Jul 10, 2025

QA Approved on Ibexa DXP Commerce 4.6-dev.

@konradoboza konradoboza merged commit 72e601c into main Jul 10, 2025
30 of 38 checks passed
@konradoboza konradoboza deleted the ibx-10170-access-check branch July 10, 2025 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@konradoboza konradoboza konradoboza approved these changes

@juskora juskora juskora approved these changes

@adamwojs adamwojs Awaiting requested review from adamwojs

@mikadamczyk mikadamczyk Awaiting requested review from mikadamczyk

@Steveb-p Steveb-p Awaiting requested review from Steveb-p

@alongosz alongosz Awaiting requested review from alongosz

@ciastektk ciastektk Awaiting requested review from ciastektk

@barw4 barw4 Awaiting requested review from barw4

@wiewiurdp wiewiurdp Awaiting requested review from wiewiurdp

Assignees

@mikadamczyk mikadamczyk

@ciastektk ciastektk

@dew326 dew326

@ViniTou ViniTou

Labels

QA approved

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@ViniTou @juskora @adamwojs @konradoboza @mikadamczyk @ciastektk @dew326