Skip to content

#335 Moved raw field value plugins and deriviers to graphql_core and …#337

Merged
pmelab merged 7 commits intodrupal-graphql:8.x-3.xfrom
blazeyo:335-move-raw-value-fields-to-graphql-core
Oct 9, 2017
Merged

#335 Moved raw field value plugins and deriviers to graphql_core and …#337
pmelab merged 7 commits intodrupal-graphql:8.x-3.xfrom
blazeyo:335-move-raw-value-fields-to-graphql-core

Conversation

@blazeyo
Copy link
Copy Markdown
Contributor

@blazeyo blazeyo commented Sep 28, 2017

…renamed them to EntityField.

First attempt. Let's see how many tests fail.

@pmelab
Copy link
Copy Markdown
Contributor

pmelab commented Oct 9, 2017

I'm hesitant merging this without #340. But could you just exclude fields that are configured with the RawValue formatter from DisplayedFieldDeriver? Then these fields would just remain the same.

@blazeyo
Copy link
Copy Markdown
Contributor Author

blazeyo commented Oct 9, 2017

Sure, added.

@pmelab
Copy link
Copy Markdown
Contributor

pmelab commented Oct 9, 2017

Awesome thanks! Merging!

I think highest priority next step is #343. If the schema is filled, we deal with slimming it down by permissions.

@blazeyo
Copy link
Copy Markdown
Contributor Author

blazeyo commented Oct 9, 2017

Cool. Pushed small fix for the notice messages. Moving to #343 once this one is green.

@pmelab pmelab merged commit ade2981 into drupal-graphql:8.x-3.x Oct 9, 2017
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