- Issue created by @omarlopesino
- Merge request !115Issue #3509343: Support all Comment entity type fields β (Merged) created by omarlopesino
- π¦πΊAustralia almunnings Melbourne, π¦πΊ
Ahoyhoy
This should already be a part of the GraphQL Compose Comment module. - πͺπΈSpain omarlopesino
Ups my bad, I didn't check the submodules. This is already supported by the graphql comments submodule, thanks for letting me know.
Would it be good idea supporting the rest of fields of the comment entity type? I mean, the entity_id, entity_type, field_name , and pid.
I've updated the MR updating the Comment GraphQLEntityType plugin with those fields.
- π¦πΊAustralia almunnings Melbourne, π¦πΊ
Back at you to take a look.
This should add the remaining fields and switch out between uuid and id on the entityId field.Caveat may be that accessing the entity->uuid() will load that entity - which could be addressed in π¬ How to return a simple object on dynamic_entity_reference values Active
- Status changed to Needs review
about 1 month ago 12:16am 6 May 2025 -
almunnings β
committed 3b990890 on 2.4.x authored by
omarlopesino β
Issue #3509343: Support all Comment entity type fields
-
almunnings β
committed 3b990890 on 2.4.x authored by
omarlopesino β
- π¦πΊAustralia almunnings Melbourne, π¦πΊ
Dang it. Didnβt see that pid field sneak in. Thatβll need to be changed to an entity reference id too.
-
almunnings β
committed c9616d61 on 2.4.x
Issue #3509343 by almunnings, omarlopesino: Support all Comment entity...
-
almunnings β
committed c9616d61 on 2.4.x
Automatically closed - issue fixed for 2 weeks with no activity.