Skip to content

Conversation

@Sreini
Copy link
Contributor

@Sreini Sreini commented Sep 23, 2025

…to prepare for deprecation of GET with body from API.

…to prepare for deprecation of GET with body from API.
@Sreini Sreini requested a review from Copilot September 23, 2025 08:32
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Tinify API client to replace HTTP GET requests with request body with POST requests to prepare for the deprecation of GET with body from the API. The change maintains existing functionality while adapting to upcoming API changes.

  • Modified the result() method to conditionally use GET (when no commands) or POST (when commands are present)
  • Updated all test files to mock POST requests instead of GET requests when a request body is involved
  • Incremented package version to 1.8.2 and added changelog entry

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/tinify/Source.ts Updated result() method to use POST when commands are present, GET when no commands
test/tinify-test.js Updated test mocks from GET to POST for requests with body
test/tinify-source-test.js Updated test mocks from GET to POST for requests with body
package.json Version bump to 1.8.2
CHANGES.md Added changelog entry for version 1.8.2

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Sreini Sreini merged commit 0e2afcf into master Sep 24, 2025
54 checks passed
@rkoopmans rkoopmans deleted the feature/remove-get-with-body branch September 24, 2025 07:59
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.

3 participants