Skip to content

Conversation

@CocoJr
Copy link
Contributor

@CocoJr CocoJr commented Jul 2, 2018

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Documented? yes
Fixed tickets #349
License MIT

Use annotation to configure the GraphQL schemas.

/**
* @\Overblog\GraphQLBundle\Annotation\GraphQLDescription(description="Released in 1977.")
*/
const NEWHOPE = 4;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Doctrine annotation not work with const

Copy link
Contributor

@mcg-web mcg-web left a comment

Choose a reason for hiding this comment

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

Thank you for contribution! Here some minors feedback to start since this is just a wip.

@@ -0,0 +1,19 @@
<?php
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the need of this file?

* @var string
*/
public $method;
} No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you please give some more ❤️ to PHP CS 😆

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Of course, bad configuration of my new computer :)

<?php
/**
* @author Thibault Colette <thibaultcolette06@hotmail.fr>
* @copyright 2018 Thibault Colette
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you please remove the copyright statement?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No problem ;)

@CocoJr
Copy link
Contributor Author

CocoJr commented Jul 16, 2018

@mcg-web Yes it's just a WIP, you can found complete example here: https://github.com/CocoJr/docker-sf4-elasticsearch-graphql-react-express

I use a boilerplate project to develop this feature.

@CocoJr CocoJr closed this Jul 29, 2018
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