Skip to content

Conversation

julitafalcondusza
Copy link
Contributor

@julitafalcondusza julitafalcondusza commented Feb 25, 2025

Question Answer
JIRA Ticket (https://github.com/ibexa/taxonomy/pull/313), (https://github.com/ibexa/taxonomy/pull/38), (ibexa/core#467)
Versions 4.6+

Support for ContentAwareInterface in ibexa_* Twig functions described in Developer Documentation.

Checklist

  • Text renders correctly
  • Text has been checked with vale
  • Description metadata is up to date
  • Added link to this PR in relevant JIRA ticket or code PR

@julitafalcondusza julitafalcondusza marked this pull request as ready for review February 27, 2025 09:41
Copy link
Contributor

@mnocon mnocon left a comment

Choose a reason for hiding this comment

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

Your changes look good, thank you!

I have two suggestions:

  1. Please remove ContentInfo mentions where they are not supported
  2. Can you please adjust code sample include lines so that the html+twig is not visible in the Image Twig functions section (and the example is complete)? It's not part of your changes, but small enough that IMHO not worth a separate PR.
Zrzut ekranu 2025-03-3 o 10 43 16

Copy link
Contributor

@mnocon mnocon left a comment

Choose a reason for hiding this comment

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

Thanks!

@julitafalcondusza julitafalcondusza merged commit 5f3f688 into master Mar 5, 2025
4 of 6 checks passed
@julitafalcondusza julitafalcondusza deleted the IBX-9415 branch March 5, 2025 06:11
julitafalcondusza added a commit that referenced this pull request Mar 5, 2025
… described in Developer Documentation (#2638)

* contentaware support added

* fixes

* full namespaces removed

* Fixes ater review

* Description fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants