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

fix: page link field type #221

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Conversation

jasonbahl
Copy link
Contributor

What does this implement/fix? Explain your changes.

This updates the "Page Link" field type to map to the GraphQL Schema as a List of Strings by default instead of a Connection.

Does this close any currently open issues?

closes #213

Any other comments?

As-is this would be a breaking change. I need to figure out if we can do this in a non-breaking way, which would mean leaving the field type mapping to the Schema as a Connection but allowing users to opt-in to the field type resolving differently (String, list of String, etc)

@jasonbahl jasonbahl added the compat: possible break There is a possibility that this might lead to breaking changes, but not confirmed yet label Jun 25, 2024
@jasonbahl jasonbahl self-assigned this Jun 25, 2024
- update PageLinkFieldTest to test against pretty permalinks
@benb1983
Copy link

benb1983 commented Oct 3, 2024

Any update when this will be fixed @jasonbahl ? I'm running into this issue now, workaround for me is a text field and getting users to enter a slug, not ideal but works for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compat: possible break There is a possibility that this might lead to breaking changes, but not confirmed yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Page Link field type should return a string, NOT a connection
2 participants