Skip to content

allow for passing object keys to searchopts to allow pagination#2303

Merged
isaacs merged 4 commits intorelease/v7.1.1from
nlf/search-opts
Dec 8, 2020
Merged

allow for passing object keys to searchopts to allow pagination#2303
isaacs merged 4 commits intorelease/v7.1.1from
nlf/search-opts

Conversation

@nlf
Copy link
Copy Markdown
Contributor

@nlf nlf commented Dec 8, 2020

our search endpoint wants the query parameter ?from=2, for example, in order to do offsets for pagination purposes. we currently have no way to express that.

this is the shortest path to resolving that problem, by allowing users to pass --searchopts=from=2, parsing it as a query parameter, and then passing that resulting object to libnpmsearch

there's a corresponding pull request for libnpmsearch to ensure that this nested opts parameter gets used.

References

Related to npm/libnpmsearch#6

@nlf nlf requested a review from a team as a code owner December 8, 2020 22:05
@darcyclarke darcyclarke added Release 7.x work is associated with a specific npm 7 release release: next These items should be addressed in the next release labels Dec 8, 2020
isaacs and others added 4 commits December 8, 2020 15:48
@isaacs
ini@1.3.6
7ceb5b7 
Fixes prototype pollution issue
@isaacs
make-fetch-happen@8.0.11
77c6ced
@isaacs
libnpmsearch@3.1.0
0ef25b6 
Update to accept query params as options, so we can paginate.
@nlf @isaacs
allow for passing object keys to searchopts to allow pagination
3db90d9 
PR-URL: #2303
Credit: @nlf
Close: #2303
Reviewed-by: @isaacs
@isaacs isaacs changed the base branch from latest to release/v7.1.1 December 8, 2020 23:57
@isaacs isaacs merged commit 3db90d9 into release/v7.1.1 Dec 8, 2020
@ruyadorno ruyadorno mentioned this pull request Dec 9, 2020
Closed
@darcyclarke darcyclarke mentioned this pull request Dec 11, 2020
Closed
@nlf nlf deleted the nlf/search-opts branch March 28, 2022 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@darcyclarke darcyclarke darcyclarke approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

release: next These items should be addressed in the next release Release 7.x work is associated with a specific npm 7 release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nlf @darcyclarke @isaacs