Skip to content
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.

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

IBX-6827: Aggregation API improvements #56

Merged
merged 3 commits into from
Dec 12, 2023
Merged

IBX-6827: Aggregation API improvements #56

merged 3 commits into from
Dec 12, 2023

Conversation

adamwojs
Copy link
Member

@adamwojs adamwojs commented Nov 1, 2023

Warning

Rebase and remove TMP commit after merging ibexa/core#287

Question Answer
JIRA issue IBX-6827
Type feature
Target Ibexa version v4.6
BC breaks no

See ibexa/core#287

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Provided automated test coverage.
  • Checked that target branch is set correctly (main for features, the oldest supported for bugs).
  • Ran PHP CS Fixer for new PHP code (use $ composer fix-cs).
  • Asked for a review (ping for example @ibexa/php-dev for back-end changes and/or @ibexa/javascript-dev for front-end changes).

Copy link

sonarcloud bot commented Nov 1, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@konradoboza konradoboza requested a review from a team December 11, 2023 07:19
@alongosz alongosz requested a review from a team December 11, 2023 13:12
@adamwojs adamwojs merged commit 0acd506 into main Dec 12, 2023
15 checks passed
@adamwojs adamwojs deleted the ibx_6827 branch December 12, 2023 12:50
Copy link

sonarcloud bot commented Dec 12, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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.

5 participants