search

SurajKharkwal / juit-time-table

College time table website
https://juit-timetable.vercel.app
MIT License
0 stars 1 forks source link

Doesn't output data for B.Tech 7th Sem. #20

Open akshit-a9 opened 2 months ago

akshit-a9 commented 2 months ago

Hey guys!

Was told about this from a junior, but the search function doesnt work for CS74, or any other batch I tried for B.Tech 7th Sem.

shorya-1012 commented 1 month ago

@akshit-a9 Thank you for providing feedback and sorry for late reply but we are looking into the problem

SurajKharkwal commented 1 month ago

@akshit-a9 Thank you for bringing this issue to my attention. Upon further review of the Excel sheet, I noticed that it contains only a single cell with the value "CS74." Additionally, there are entries such as "CSALL," which I believe represent all batches in the CS department.

It appears that the current implementation is filtering specifically for the "CS74" batch, which explains why only one result is being retrieved. To resolve this, I plan to implement a separate API that will handle batch filtering for the 7th semester specifically and also account for entries like "CSALL"

If you have any suggestions or improvements, I welcome feedback and contributions. Please feel free to open a PR if you'd like to make any changes.

Thank you again for your assistance.