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-7653: IsContainer criterion added #333

Merged
merged 15 commits into from
May 16, 2024

Conversation

tischsoic
Copy link
Contributor

Question Answer
JIRA issue https://issues.ibexa.co/browse/IBX-7653
Type improvement/bug
Target Ibexa version v4.6
BC breaks yes/no

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 @ibexa/engineering).

@tischsoic tischsoic self-assigned this Feb 12, 2024
@tischsoic tischsoic force-pushed the IBX-7653-images-visible-in-tree-browser-in-image-picker branch 5 times, most recently from a9cf699 to bdb8ced Compare February 22, 2024 11:10
@tischsoic tischsoic marked this pull request as ready for review February 22, 2024 11:27
@tischsoic tischsoic requested a review from a team February 22, 2024 11:28
@adamwojs
Copy link
Member

adamwojs commented Mar 7, 2024

@tischsoic You are missing Solr and Elasticsearch counterpart PRs

@tischsoic tischsoic changed the base branch from main to 4.6 March 25, 2024 10:18
@tischsoic
Copy link
Contributor Author

I added solr and elasticsearch support for the IsContainer criterion.

@tischsoic tischsoic force-pushed the IBX-7653-images-visible-in-tree-browser-in-image-picker branch from d8a24ff to d394a16 Compare March 25, 2024 11:32
@Steveb-p Steveb-p requested a review from a team April 4, 2024 12:24
@tischsoic tischsoic requested a review from Steveb-p April 5, 2024 07:59
@tischsoic tischsoic requested a review from konradoboza April 18, 2024 08:14
Copy link
Member

@alongosz alongosz left a comment

Choose a reason for hiding this comment

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

CI is failing here

Seems we have outdated Sonar Cloud exclusion path here https://github.com/ibexa/core/blob/4.6/.sonarcloud.properties.

@tischsoic tischsoic force-pushed the IBX-7653-images-visible-in-tree-browser-in-image-picker branch 4 times, most recently from 3f60f79 to f08c85f Compare April 18, 2024 10:05
@tischsoic tischsoic force-pushed the IBX-7653-images-visible-in-tree-browser-in-image-picker branch from f4b88a0 to cbbfcbf Compare May 15, 2024 09:32
@tischsoic tischsoic force-pushed the IBX-7653-images-visible-in-tree-browser-in-image-picker branch from cbbfcbf to eb52380 Compare May 15, 2024 12:51
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@dew326 dew326 merged commit 4bcf0c5 into 4.6 May 16, 2024
21 checks passed
@dew326 dew326 deleted the IBX-7653-images-visible-in-tree-browser-in-image-picker branch May 16, 2024 07:21
@tischsoic
Copy link
Contributor Author

Merged up:

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.

8 participants