Bug: Raycast Extension not showing all links (need pagination)

[steven-tey]

More details here: raycast/extensions#14198

[linear]

ENG-852 Bug: Raycast Extension not showing all links (need pagination)

[DeepaPrasanna]

/assign

[oss-gg]

Assigned to @DeepaPrasanna! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

[naaa760]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[Kartikey1306]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[DeepaPrasanna]

I really wanted to work on this but as this requires raycast running locally which only runs on macOS, I am unassigning myself. I dont have macOs.

[Rishi-0007]

/assign

[oss-gg]

Assigned to @Rishi-0007! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

[Ionfinisher]

Hey @steven-tey Hope you're doing well. As there was no draft PR for this issue within the 48h+ I've been working already on a solution for this issue. Tomorrow, I'll share my progress on this and ask for feedback if that's okay

[steven-tey]

@Ionfinisher sounds good! reassigned to you!

[Ionfinisher]

Hey @steven-tey, sorry for the late response. Had a little more work today than I tought I would. Here is my update so far.

• Fist I refactored the getAllShortLinks function in the api/index.ts file for the use typeahead with the search param as mentioned in #14198 . As the dub API doesn't acctually provide a way to know if there's a next page when getting when the paginated response, I tryed coming up with a solution but not so efficient for the hasMoreLinks.

• Updated the useShortLinks hook to dynamically accept a query parameter, so that different search queries can be passed as arguments.

• Updated the List to dynamically trigger typeahead when a page cursor is present for the next page of results. But if the number of links is < 100 then it uses keywords and use filtering=true

• Also just encountered a type error from the mutate in deleteLink function. Trying to solve it now and any suggestions and feedbacks for improvement are welcomed

[steven-tey]

@Ionfinisher great work! can you link to the PR?

[Ionfinisher]

Hey @steven-tey here is the PR: raycast/extensions#15162