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

Update "query parser" default value in 5.x #1425

Merged
merged 1 commit into from
May 26, 2024
Merged

Update "query parser" default value in 5.x #1425

merged 1 commit into from
May 26, 2024

Conversation

krzysdz
Copy link
Contributor

@krzysdz krzysdz commented Jun 7, 2023

"query parser" defaults to "simple" since 5.0.0-beta.1, but 9ce25eb did not update the value in app-settings.md.

"query parser" defaults to `"simple"` since 5.0.0-beta.1, but 9ce25eb did not update the value in `app-settings.md`.
@crandmck crandmck added the 5.x Docs for 5.x version label Mar 5, 2024
@crandmck crandmck added the needs tech review A doc edit that requires technical review before merging label May 24, 2024
@krzysdz
Copy link
Contributor Author

krzysdz commented May 26, 2024

@crandmck crandmck self-assigned this May 26, 2024
@crandmck crandmck self-requested a review May 26, 2024 18:31
@crandmck crandmck merged commit 72ad1fd into expressjs:gh-pages May 26, 2024
1 check passed
@crandmck
Copy link
Member

Thanks for those references @krzysdz, that made it much easier for me to review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5.x Docs for 5.x version needs tech review A doc edit that requires technical review before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants