FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Bump 2.4.1: safer pagination and aggregation by cigolpl · Pull Request #152 · itemsapi/itemsjs · GitHub

/ itemsjs Public

Bump 2.4.1: safer pagination and aggregation - #152

Merged
cigolpl merged 1 commit into
masterfrom
chore/2.4.1-pagination
Nov 25, 2025
ConversationCommits (1)ChecksFiles changed
Merged

Bump 2.4.1: safer pagination and aggregation#152
cigolpl merged 1 commit into
masterfrom
chore/2.4.1-pagination

Conversation

cigolpl commented Nov 25, 2025

Copy link
Copy Markdown
Member

Summary

  • normalize pagination inputs (NaN/Infinity/negative) to safe defaults, keep per_page=0 behavior
  • avoid mutating aggregation config when using aggregation(), guard missing not_filters and similar id edge cases
  • bump version to 2.4.1 with tests covering new behaviors

Testing

  • npm test

cigolpl merged commit 516f614 into master Nov 25, 2025
2 checks passed
cigolpl deleted the chore/2.4.1-pagination branch November 25, 2025 07:24
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

Back | FazBrowse Home | New Git URL