Skip to content

fix: update search all file logic with pageToken #351

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

QuocThuanTruong
Copy link

After reviewing our official document, I noticed that the logic in the Node.js tab is not in line with the logic used in other languages. This creates confusion for readers like myself. That's why I am proposing this update, aimed at making the logic more clear and comprehensive. To achieve this, I am utilizing the pageToken and files variables in a more effective manner.

@QuocThuanTruong
Copy link
Author

Hello @jpoehnelt, could you please let me know the status of my PR review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants