Support application/vnd.api+json content type

jellydn / hurl.nvim

Hurl.nvim is a Neovim plugin designed to run HTTP requests directly from `.hurl` files. Elevate your API development workflow by executing and viewing responses without leaving your editor.

https://gyazo.com/19492e8b5366cec3f22d5fd97a63f37a

MIT License

157 stars 15 forks source link

Support application/vnd.api+json content type #50

Closed andrevdm closed 10 months ago

andrevdm commented 10 months ago

WHAT

Detect JSON:API content-type

WHY

JSON:API returns valid JSON but uses a different content type. Update the is_json_response function to handle both cases

HOW

Match "content-type: application/vnd.api+json"

Types of changes

[X] New feature (non-breaking change which adds functionality)

changeset-bot[bot] commented 10 months ago

⚠️ No Changeset found

Latest commit: ac21d10a80e405a19a7a568f7c37a93d127bb7da

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

sweep-ai[bot] commented 10 months ago

Apply Sweep Rules to your PR?

[ ] Apply: All docstrings and comments should be up to date.
[ ] Apply: Commit messages should be descriptive and provide a clear explanation of the changes made.
[ ] Apply: Author names should be consistent and use proper formatting.
[ ] Apply: Code should be properly formatted and follow a consistent style guide.
[ ] Apply: There should be no unnecessary whitespace or trailing spaces in the code.
[ ] Apply: Variable and function names should be meaningful and descriptive.

jellydn commented 10 months ago

LGTM! Thanks.

jellydn commented 10 months ago

@all-contributors add @andrevdm for code.

allcontributors[bot] commented 10 months ago

@jellydn

I've put up a pull request to add @andrevdm! :tada: