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

[FR] Let graphql transform directive apply to all asset properties #5425

Closed
membla opened this issue Jan 12, 2020 · 1 comment
Closed

[FR] Let graphql transform directive apply to all asset properties #5425

membla opened this issue Jan 12, 2020 · 1 comment
Assignees
Milestone

Comments

@membla
Copy link

@membla membla commented Jan 12, 2020

Description

As far as I can tell, the transform directive can only be applied to the url field of the asset in graphql. At a minimum, it should also be able to give the correct path of the transform, but more intuitively it should work on the asset field and return all properties of the given transform, e.g.

image @transform(handle:"largeFrontImage") {
url
path
width
height
}

Should return the properties of the transformed asset.

Additional info

  • Craft version: 3.4.beta-5
@membla membla changed the title [FR] Let graphql transform directive apply to all asset fields [FR] Let graphql transform directive apply to all asset properties Jan 12, 2020
andris-sevcenko added a commit that referenced this issue Jan 16, 2020
…ire Asset element.

Resolve #5425
@andris-sevcenko

This comment has been minimized.

Copy link
Member

@andris-sevcenko andris-sevcenko commented Jan 16, 2020

Great idea! Just pushed this to the 3.4 branch.

@brandonkelly brandonkelly added this to the 3.4 milestone Jan 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.