Skip to content

fix(upcoming): answer an empty list instead of 404 when nothing is up… - #569

Open
Dx-Alz-xD wants to merge 1 commit into
CodeChefVIT:stagingfrom
Dx-Alz-xD:krish/upcoming-empty-list
Open

Dx-Alz-xD wants to merge 1 commit into
CodeChefVIT:stagingfrom
Dx-Alz-xD:krish/upcoming-empty-list

Conversation

@Dx-Alz-xD

Copy link
Copy Markdown

📌 Purpose

With no upcoming exams, /api/upcoming-papers returned 404, which filled the browser console with errors.


Corresponding issue: closes #549


🖼️ Showcase

With nothing scheduled, the endpoint returns 200 [], and the home page shows "No more upcoming papers, Enjoy your break!" with no errors in the console.


🔧 Changes

  • The route returns the (possibly empty) list with 200

➕ Additional Notes

  • The home carousel and the request page already handle an empty list

…coming

between exams the endpoint answered 404, which filled the browser
console with errors. the client already shows a message for an empty
list

This branch has not been deployed

No deployments
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.

/api/upcoming-papers Throws 404 When No Exams are Active

1 participant