Skip to content

Update search.json default fields to be more restrictive #10350

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Jan 16, 2025
Merged

Update search.json default fields to be more restrictive #10350

merged 3 commits into from
Jan 16, 2025

Conversation

Image for: Conversation
Copy link
Member

Reduce default response load for search.json

Patrons can still use fields=* to achieve the old results

Related to #7673

Technical

Returns a documentation_url which ideally would return json but 🤷

Testing

hit search.json and you should get less payload

Screenshot

Stakeholders
mekarpeles and others added 2 commits January 16, 2025 13:52
Update code.py
1a56f24 
Reduce default response load for search.json

Patrons can still use `fields=*` to achieve the old results
[pre-commit.ci] auto fixes from pre-commit.com hooks
81270c9 
for more information, see https://pre-commit.ci
mekarpeles added the Priority: 1 Do this week, receiving emails, time sensitive, . [managed] label Jan 16, 2025
mekarpeles merged commit 51b3a79 into master Jan 16, 2025
7 checks passed
cdrini deleted the fix/seachjson branch March 3, 2025 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 1 Do this week, receiving emails, time sensitive, . [managed]
3 participants