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

Bug: findSurround didn't respect the sort, only parameter #2337

Closed
N-index opened this issue Sep 24, 2023 · 2 comments
Closed

Bug: findSurround didn't respect the sort, only parameter #2337

N-index opened this issue Sep 24, 2023 · 2 comments

Comments

@N-index
Copy link

N-index commented Sep 24, 2023

Environment


  • Operating System: Windows_NT
  • Node Version: v18.17.1
  • Nuxt Version: 3.7.0
  • CLI Version: 3.7.2
  • Nitro Version: 2.6.2
  • Package Manager: npm@9.6.7
  • Builder: -
  • User Config: app, modules, dayjs, content
  • Runtime Modules: @nuxt/content@2.8.2, @unocss/nuxt@0.56.1, @vueuse/nuxt@10.4.1, dayjs-nuxt@1.1.2, @formkit/auto-animate/nuxt@0.8.0
  • Build Modules: -

Reproduction

https://stackblitz.com/edit/github-ojj9sc?file=content%2Findex.md

Describe the bug

  • findSurround not respect the sort parameter, it just obeys the alphabet order of filename.
  • findSurround not respect the only parameter, it returns every field of the article.
  • this issue happens both on queryContent and ContentQuery component.

Additional context

No response

Logs

No response

@nobkd
Copy link
Contributor

nobkd commented Sep 24, 2023

Probably already fixed on edge channel with #2311.
Can you check?

@N-index
Copy link
Author

N-index commented Sep 25, 2023

Sorry, I just searched findSurround keyword in existing issues, and didn't see #2311, so duplicate open this new issue.
Now I checked the edge channel and it has been fixed.

@N-index N-index closed this as completed Sep 25, 2023
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

No branches or pull requests

2 participants