Skip to content

Conversation

GDefrevillePEReN
Copy link
Contributor

No description provided.

Copy link
Member

@Cli4d Cli4d left a comment

Choose a reason for hiding this comment

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

Hi @GDefrevillePEReN

Thank you for making this contribution. I have reviewed it, and everything checks out except the service name. The service name should be as I have suggested, based on the terms you added. This links with your previous contribution in #612.

Then the change will reflect in the JSON file, and also the name of the JSON file as the Service ID is the same as the Service name.

You can make the changes and request a review, and I will proceed to merge it. Thank you

@@ -0,0 +1,12 @@
{
"name": "My Volkswagen",
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"name": "My Volkswagen",
"name": "Volkswagen Group France",

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