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

[Feat]: Introducing Pagination Support for Notion API in Page and Database Responses #44

Closed
wants to merge 0 commits into from

Conversation

brf153
Copy link
Contributor

@brf153 brf153 commented Jan 21, 2024

Issue(s)

closes #38

Acceptance Criteria fulfillment

  • Check how the pagination is working in the api
  • Make changes to searchPage and searchDatabase methods.

Proposed changes (including videos or screenshots):

Made changes to searchPage method. Now, we are getting all the pages made in a workspace.
Screenshot of the length of the final results after concatenating all the response.data.results
Screenshot from 2024-01-21 21-36-14

@brf153 brf153 changed the title [ Feat ]: Added recursive function for searchPages [ Feat ]: Added recursive function for Pages and Databases Jan 22, 2024
@brf153 brf153 changed the title [ Feat ]: Added recursive function for Pages and Databases [Feat]: Introducing Pagination Support for Notion API in Page and Database Responses Jan 22, 2024
@brf153
Copy link
Contributor Author

brf153 commented Jan 22, 2024

@Nabhag8848 I have implemented changes that enable paginated responses for searchPages, searchDatabases, and searchPagesAndDatabases. Also, I need your help. I have successfully retrieved all the pages and databases from the response (verified in the terminal). However, I am facing an issue: when making a /notion create, /notion create db, or /notion share request, the dialog box does not appear on the screen. I'm unsure if this is a server error or an issue with my system. Could you please review these changes on your system? I made modifications in the Notion SDK file. Apologies for any inconvenience caused.

@brf153 brf153 marked this pull request as ready for review January 22, 2024 18:31
@brf153
Copy link
Contributor Author

brf153 commented Jan 23, 2024

I don't know how but it has started working in the morning. Now we are getting all the pages and the databases. I will send a video for it.

@brf153
Copy link
Contributor Author

brf153 commented Jan 23, 2024

WhatsApp.Video.2024-01-23.at.10.13.26.mp4

@brf153 brf153 marked this pull request as draft January 23, 2024 08:01
@brf153
Copy link
Contributor Author

brf153 commented Jan 24, 2024

WhatsApp.Video.2024-01-24.at.10.30.00.mp4
WhatsApp.Video.2024-01-24.at.10.04.29.mp4

@brf153 brf153 marked this pull request as ready for review January 24, 2024 05:03
@brf153
Copy link
Contributor Author

brf153 commented Jan 24, 2024

@Nabhag8848 I've implemented the changes, and while there might be occasional delays in responses, we are consistently receiving them. I'm not sure if it's a system issue or another factor. Could you please check it on your end? I appreciate your understanding, and I'm confident we'll resolve any minor hiccups.

@Nabhag8848 Nabhag8848 added this to Pending Review in Notion App - Community Dashboard Jan 31, 2024
@Nabhag8848
Copy link
Collaborator

@brf153 Can you resolve merge conflicts as i merged your, remove emoji pr.

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

Successfully merging this pull request may close these issues.

[Bug] Search of Pages and / or Database in SearchComponent Currently Doesn't Support Pagination
2 participants