fix: Set default size for GET /extension-requests API

[AnujChhikara]

Date: 7 Feb 2025

Developer Name: @AnujChhikara

---

Issue Ticket Number

• Define Default Size for extension-requests API to Prevent Returning All Data #2338

Description

• Added a default size=5 for the /extension-requests API when the size query parameter is not provided.
• Wrote a test to verify that the API returns exactly 5 extension requests by default.

Documentation Updated?

• Yes
• No

Under Feature Flag

• Yes
• No

Database Changes

• Yes
• No

Breaking Changes

• Yes
• No

Development Tested?

• Yes
• No

Screenshots

Screenshot 1

• In the video, you can see that if the size query parameter is not provided, the response returns only five objects.

Postman_9k6RGKSYrs.mp4

Screencast.2025-02-12.21.59.04.mp4

Test Coverage

Screenshot 1

Additional Notes

[vikasosmium]

is this video a GIF in screenshots 1?

not able to pause it to see the response.

[AnujChhikara]

is this video a GIF in screenshots 1?

not able to pause it to see the response.

It's a gif, changing it with a video

[shobhan-sundar-goutam]

1. Why are we only testing in postman? Why not in actual dashboard site?
2. Why this is not under feature flag?

[AnujChhikara]

1. Why are we only testing in postman? Why not in actual dashboard site?
2. Why this is not under feature flag?

• The issue being discussed is specifically related to the API response, which is why we are testing it in Postman. However, to ensure complete verification, I have also included a video demonstration of the dashboard site. In the video, you can see that when no size query parameter is provided, the API returns only 5 requests by default. As we scroll down, additional requests are fetched dynamically.
• This change does not require a feature flag because size=5 is already the default behavior on the extension-requests page. You can refer to the relevant code here: Constants.js. The only issue is that the page size is determined from the URL. If we remove size=5 from the URL, the page currently loads all requests at once, which is not the expected behavior. With these changes, even if the size=5 query parameter is removed from the URL, the default behavior remains intact, ensuring limited API requests.