Skip to content

Commit

Permalink
Merge pull request #11 from ConductionNL/development
Browse files Browse the repository at this point in the history
camelCase to snake_case
  • Loading branch information
rubenvdlinde authored Dec 19, 2019
2 parents 98e9920 + 356ec69 commit 093b417
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 10 deletions.
6 changes: 3 additions & 3 deletions INSTALLATION.md
Original file line number Diff line number Diff line change
Expand Up @@ -113,9 +113,9 @@ $ helm install --name vrc-prod ./api/helm --kubeconfig="api/helm/kubeconfig.yaml

Or update if you want to update an existing one
```CLI
$ helm upgrade pc-dev ./api/helm --kubeconfig="api/helm/kubeconfig.yaml" --namespace=dev --set settings.env=dev,settings.debug=1
$ helm upgrade pc-stag ./api/helm --kubeconfig="api/helm/kubeconfig.yaml" --namespace=stag --set settings.env=stag,settings.debug=0
$ helm upgrade pc-prod ./api/helm --kubeconfig="api/helm/kubeconfig.yaml" --namespace=prod --set settings.env=prod,settings.debug=0
$ helm upgrade vrc-dev ./api/helm --kubeconfig="api/helm/kubeconfig.yaml" --namespace=dev --set settings.env=dev,settings.debug=1
$ helm upgrade vrc-stag ./api/helm --kubeconfig="api/helm/kubeconfig.yaml" --namespace=stag --set settings.env=stag,settings.debug=0
$ helm upgrade vrc-prod ./api/helm --kubeconfig="api/helm/kubeconfig.yaml" --namespace=prod --set settings.env=prod,settings.debug=0
```

Or del if you want to delete an existing one
Expand Down
2 changes: 1 addition & 1 deletion api/config/packages/api_platform.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ api_platform:
enable_re_doc: true

# In the NL api strategie we use snake case for naming
# name_converter: 'Symfony\Component\Serializer\NameConverter\CamelCaseToSnakeCaseNameConverter'
name_converter: 'Symfony\Component\Serializer\NameConverter\CamelCaseToSnakeCaseNameConverter'

# Lets confirm to JSON-HA: first, as per common ground. But allow other serializers
formats:
Expand Down
5 changes: 5 additions & 0 deletions api/helm/templates/php-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,11 @@ spec:
configMapKeyRef:
name: {{ template "fullname" . }}
key: app-domain
- name: APP_SUBDOMAIN
valueFrom:
configMapKeyRef:
name: {{ template "fullname" . }}
key: app-domain
# organization
- name: ORGANIZATION_NAME
valueFrom:
Expand Down
9 changes: 3 additions & 6 deletions api/helm/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,15 +10,12 @@ settings:
version: V.0.1
description: ''Naast deze JSON rest API is er ook een [graphql](/graphql) interface beschikbaar.''
repro: 'https://github.com/ConductionNL/Proto-component-commonground'
subdomain: pc
domain: conduction.nl
domains:
- conduction.nl
# - vng.cloud
# - zaakonline.nl
# - larping.eu
# - common-ground.dev
# - trouwplanner.online
- zaakonline.nl
- huwelijksplanner.online
- common-ground.dev
organisationName: Conduction
email: [email protected]
country: Netherlands
Expand Down

0 comments on commit 093b417

Please sign in to comment.