Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: remove deprecated methods for paging, range and limit #398

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

TiagoGrosso
Copy link
Owner

The methods addPaging(), addRange() and addLimit() have been deprecated for a long time and it's time to remove them.

BREAKING CHANGE: addPaging(), addRange() and addLimit() are no longer available and should be replaced with withPaging(), withRange() and withLimit()

The methods `addPaging()`, `addRange()` and `addLimit()` have been
deprecated for a long time and it's time to remove them.

BREAKING CHANGE: `addPaging()`, `addRange()` and `addLimit()` are no
longer available and should be replaced with `withPaging()`,
`withRange()` and `withLimit()`
@TiagoGrosso TiagoGrosso merged commit 560100b into master Aug 8, 2024
6 checks passed
@TiagoGrosso TiagoGrosso deleted the remove-deprecated-add-paging-add-range branch August 8, 2024 22:47
TiagoGrosso pushed a commit that referenced this pull request Aug 8, 2024
# [7.0.0](v6.2.0...v7.0.0) (2024-08-08)

### Bug Fixes

* **docs:** update readme after axios removal ([#397](#397)) ([2987796](2987796))
* **it:** remove fetch mock from integration tests ([#399](#399)) ([e8ae3c3](e8ae3c3))

### chore

* remove deprecated methods for paging, range and limit ([#398](#398)) ([560100b](560100b))

### Features

* **request:** remove axios and use native fetch instead ([#396](#396)) ([6036fea](6036fea))

### BREAKING CHANGES

* `addPaging()`, `addRange()` and `addLimit()` are no
longer available and should be replaced with `withPaging()`,
`withRange()` and `withLimit()`
* **request:** Since Axios has been removed, error handling with
`AxiosError` will no longer work and should be done in accordance with
the native Node `fetch`.
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.

1 participant