Skip to content

[ACTION] Make sure MS Outlook actions are paginating / returning all results #16233

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

Closed
dannyroosevelt opened this issue Apr 9, 2025 · 2 comments · Fixed by #16255
Closed
Assignees
Labels
action New Action Request enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed triaged For maintainers: This issue has been triaged by a Pipedream employee

Comments

@dannyroosevelt
Copy link
Collaborator

It seems like we might be limiting the number of folders we're returning in the folderId prop for microsoft_outlook (looks like a max of 10 items maybe). We should make sure we're paginating here (and all other async options props for the app).

@dannyroosevelt dannyroosevelt added action New Action Request enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Apr 9, 2025
@dannyroosevelt dannyroosevelt added the triaged For maintainers: This issue has been triaged by a Pipedream employee label Apr 9, 2025
@michelle0927 michelle0927 self-assigned this Apr 10, 2025
@michelle0927 michelle0927 moved this from Prioritized to Doing in Component (Source and Action) Backlog Apr 10, 2025
@michelle0927 michelle0927 moved this from Doing to Ready for PR Review in Component (Source and Action) Backlog Apr 10, 2025
@luancazarine luancazarine moved this from Ready for PR Review to In Review in Component (Source and Action) Backlog Apr 11, 2025
@luancazarine luancazarine moved this from In Review to Ready for QA in Component (Source and Action) Backlog Apr 11, 2025
@vunguyenhung vunguyenhung moved this from Ready for QA to In QA in Component (Source and Action) Backlog Apr 14, 2025
@vunguyenhung vunguyenhung moved this from In QA to Changes Required in Component (Source and Action) Backlog Apr 14, 2025
@vunguyenhung
Copy link
Collaborator

Hello everyone, I have tested this PR and there're some test cases failed or needed improvement.

Please check the test report below for more information
https://vunguyenhung.notion.site/ACTION-Make-sure-MS-Outlook-actions-are-paginating-returning-all-results-1d2bf548bb5e81b9bd82fb790bd46fec

@michelle0927 michelle0927 moved this from Changes Required to Ready for QA in Component (Source and Action) Backlog Apr 14, 2025
@vunguyenhung vunguyenhung moved this from Ready for QA to In QA in Component (Source and Action) Backlog Apr 15, 2025
@vunguyenhung vunguyenhung moved this from In QA to Ready for Release in Component (Source and Action) Backlog Apr 15, 2025
@vunguyenhung
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action New Action Request enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed triaged For maintainers: This issue has been triaged by a Pipedream employee
Development

Successfully merging a pull request may close this issue.

3 participants