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 nextjs-docs extension #15303

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

danieldeichfuss
Copy link
Contributor

@danieldeichfuss danieldeichfuss commented Nov 10, 2024

Description

When using Next.js, you will usually choose one router (App or pages). Currently, the default is to show both results, which makes it easy to read the wrong docs for your use case. That's why I usually use the dropdown to filter for one Router. Before the selection was not persisted and had to be selected every time. Now it's being persisted.

Screencast

Screen.Recording.2024-11-10.at.20.23.30.yafw.balanced.mp4

Checklist

- Merge branch \'contributions/merge-1731265995833469000\'
- Pull contributions
- Persist Selected Next Router
- Fork
@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: nextjs-docs Issues related to the nextjs-docs extension labels Nov 10, 2024
@raycastbot
Copy link
Collaborator

Thank you for your contribution! 🎉

🔔 @PrasadChinwal @xmok you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

You can expect an initial review within five business days.

@PrasadChinwal
Copy link
Contributor

@danieldeichfuss Great work! Thank you so much for the contribution. Ready to be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension fix / improvement Label for PRs with extension's fix improvements extension: nextjs-docs Issues related to the nextjs-docs extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants