Skip to content
This repository has been archived by the owner on Feb 10, 2019. It is now read-only.

normalize $prefix to make it useable as schema #386

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

RTLer
Copy link

@RTLer RTLer commented Sep 19, 2018

hi
this controller uses graphql.prefix config as part of shema name but prefix can be some thing like api/graphql or something like it
i just normalize it to make it to search for api_graphql_schema instead of api/graphql_schema which isn't a valid route param name
for this instance graphql.php is :

[
    'prefix' => 'api/graphql',
    'routes' => '{api_graphql_schema?}',
]

hi
this controller uses graphql.prefix config as part of shema name but prefix can be some thing like `api/graphql` or something like it
i just normalize it to make it to search for `api_graphql_schema` instead of `api/graphql_schema` which isn't a valid route param name
for this instance graphql.php is :

```php
[
    'prefix' => 'api/graphql',
    'routes' => '{api_graphql_schema?}',
]
```
@coveralls
Copy link

Coverage Status

Coverage remained the same at 70.537% when pulling b6eafbd on RTLer:patch-1 into 66e9233 on Folkloreatelier:develop.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants