Skip to content

Conversation

mcg-web
Copy link
Contributor

@mcg-web mcg-web commented May 10, 2017

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

If not using modern or classic flag the command will default on classic behavior. If you want to change this, you can use bundle config:

overblog_graphql:
  versions:
    relay: modern

@mcg-web mcg-web requested a review from ooflorent May 10, 2017 07:37
@mcg-web mcg-web force-pushed the json-schema-dump-for-modern-relay-compiler branch from 4ea6b88 to c35d015 Compare May 11, 2017 06:34
@mcg-web mcg-web force-pushed the json-schema-dump-for-modern-relay-compiler branch from c35d015 to 38a4fc1 Compare May 11, 2017 07:56
$container->setParameter($this->getAlias().'.versions.react', $config['versions']['react']);
$container->setParameter($this->getAlias().'.versions.fetch', $config['versions']['fetch']);
foreach ($config['versions'] as $key => $version) {
$container->setParameter($this->getAlias().'.versions.'.$key, $version);
Copy link
Contributor

Choose a reason for hiding this comment

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

Use sprintf please

@mcg-web mcg-web merged commit ddaabdc into overblog:master May 11, 2017
@mcg-web mcg-web deleted the json-schema-dump-for-modern-relay-compiler branch May 11, 2017 08:29
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