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

BUGFIX: Allow Enum fields in input types #413

Conversation

unclecheese
Copy link

Regression from the DBFieldTypes work. Resolves #405

Copy link
Member

@emteknetnz emteknetnz left a comment

Choose a reason for hiding this comment

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

Looks good

Couple of small things re failing travis tests:

@unclecheese
Copy link
Author

@emteknetnz The build is failing because the wrong version of framework test is being used. The 0.4 release uses the correct config. https://github.com/silverstripe/silverstripe-frameworktest/releases/tag/0.4.2

Do we need to update recipe-testing?

@unclecheese
Copy link
Author

Looks like the version of framework test is a variable now? https://app.travis-ci.com/github/silverstripe/silverstripe-graphql/jobs/546767496#L421

I'm kind of confused how the new travis setup works. Could use your advice. :)

@unclecheese unclecheese force-pushed the pulls/master/enummy-nummy-in-my-tummy branch from fc7da41 to f8dcfe3 Compare November 5, 2021 03:16
@unclecheese unclecheese force-pushed the pulls/master/enummy-nummy-in-my-tummy branch from f8dcfe3 to 91bba54 Compare November 5, 2021 03:29
@unclecheese unclecheese merged commit 23953e2 into silverstripe:master Nov 10, 2021
@unclecheese unclecheese deleted the pulls/master/enummy-nummy-in-my-tummy branch November 10, 2021 02:33
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.

2 participants