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

Enable to define a maximum value for the search limit #748

Closed
wants to merge 2 commits into from

Conversation

remicres
Copy link
Contributor

@remicres remicres commented Sep 10, 2024

Related Issue(s):

Description:

This patch intends to propose a way to set the maximum value for the search limit.

PR Checklist:

  • pre-commit hooks pass locally
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable, and docs build successfully (run make docs)
  • Changes are added to the CHANGELOG.

@vincentsarago
Copy link
Member

vincentsarago commented Sep 10, 2024

@remicres sorry for closing your issue quite fast, I didn't realized a PR was coming up 😅

I personally would love to avoid having settings for those kind of things (it seems nice for one attribute, but it could also open a pandora box).

I do believe that it's quite simple for user to use their own GET/POST models with stac-fastapi 3.0 which should solve most people demands!

@remicres
Copy link
Contributor Author

@vincentsarago no worry, it makes sense

@remicres remicres closed this Sep 10, 2024
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.

2 participants