MyBusinessQA Service has wrong paths

[unbreak]

In src/MyBusinessQA.php for locations_questions there are wrong paths for methods create and list, fot both should be:

'path' => 'v1/{+parent}/questions',

not:

'path' => 'v1/{+parent}',

Steps to reproduce

1. try to call:

$qaService = new MyBusinessQA($client);
$questions = $qaService->locations_questions->listLocationsQuestions($locationName);

2. You will get exception that read_mask is required field because its trying to get location not questions list for provided location.

[unbreak]

Fixing by #1010

[bshaffer]

The discovery doc shows the path we have in the library here as the path for listing questions. There's nothing we can do here unless that changes. I would suggest contacting the API team if you think this is an error.

The maintainers of this repo only make sure the generated clients match the API's discovery doc, and do not support any specific API usage questions (there are over 200 APIs, so it would be impossible for us to do that)

[bshaffer]

I'm looking into the flatPath parameter in the discovery doc, as that seems to be closer to the URL you've provided

Update: The flatPath parameter is not used by any of our discovery API client library generators, so we think this may be a problem in the API's discovery doc.

[bshaffer]

So, looking at the discovery doc, the parent parameter should contain "/questions" in the path:

{
"list": {
    "parameters": {
        "parent": {
            "description": "Required. The name of the location to fetch questions for.",
            "location": "path",
            "required": true,
            "type": "string",
            "pattern": "^locations/[^/]+/questions$"
        },
    }
}

So the correct usage should be:

$qaService = new MyBusinessQA($client);
$questions = $qaService->locations_questions->listLocationsQuestions('locations/LOCATION_NAME/questions');

I agree this is not very intuitive, but I believe this is how the API is designed.