fix: create clip api method and change response type

[jacobtread]

Description

Updates the create-clip API to use the POST method instead of GET along with some minor docs updates

Changes

• Updates the create-clip request to use the POST method
• Align the documentation with the latest current Twitch documentation

Related Issues

• Closes Incorrect HTTP method for clips endpoint #496

[Nerixyz]

I do wonder why we return a vector of clips rather than one. That would be a breaking change, though. So we should keep that in mind when we do a 0.x bump.

[jacobtread]

I do wonder why we return a vector of clips rather than one. That would be a breaking change, though. So we should keep that in mind when we do a 0.x bump.

The twitch API documentation here specifies that the response is an array

{
   "data":
   [{
      "id": "FiveWordsForClipSlug",
      "edit_url": "http://clips.twitch.tv/FiveWordsForClipSlug/edit"
   }]
}

Not sure if you have any way built into your response handling to transform that?

[Nerixyz]

The twitch API documentation here specifies that the response is an array

Yea, all responses from Twitch are arrays (not sure why).

Not sure if you have any way built into your response handling to transform that?

We have parse_single_return for this.

[Emilgardis]

Could we change the response type in the impl Response to just be CreatedClip, this change will need a bump anyway. See for example

twitch_api/src/helix/endpoints/users/update_user.rs

Line 94 in 7e676ec

helix::parse_single_return(request, uri, response, status)

for how to do it

[jacobtread]

I have updated it to use the parse_single_return :)

[Nerixyz]

LGTM