Remove or replace unused favorites API and middleware code
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
- Issue type
- Refactor
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- next.js, typescript
- Domain
- authentication, backend
Research direction
Read src/services/api.tsx and middleware.ts, then search the codebase for references to the three favorites helpers and the /user/favorites matcher. Remove the unused helpers and update the matcher; done when no obsolete favorites API or middleware references remain and the remaining timetable routes are still protected as intended.
Written by the indexing model from the issue text.
Description
src/services/api.tsx contains getFavourites, deleteFavourite, and renameFavourite that call /api/user/favorites. This route no longer exists (timetables now use /saved and /api/timetables). middleware.ts is also still protecting /user/favorites, and no files in the codebase import these functions.
What needs to be done:
- Delete unused favorites API helpers and update
middleware.tsmatchers.
Files: src/services/api.tsxmiddleware.ts
- Dominant language
- TypeScript
- Stars
- 6
- Forks
- 27
- Avg merge
- 1d 8h
- Merged PRs (30d)
- 14
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from CodeChefVIT/ffcs
-
Mobile saved page gives no feedback on Copy Link and hides errorsPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
CodeChefVIT/ffcs#274 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
CodeChefVIT/ffcs#273 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
CodeChefVIT/ffcs#272 ·
Maintainers usually reply within 1 day
-
PDF Download crashes silently when page is refreshedPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
CodeChefVIT/ffcs#271 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
CodeChefVIT/ffcs#261 ·
Maintainers usually reply within 1 day
All issues in CodeChefVIT/ffcs
Similar issues
-
calcite-ui-icons enhancement impact - p3 - not time sensitive needs triage
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
Esri/calcite-design-system#15467 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
diegosouzapw/OmniRoute#18390 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
Difficulty 1/5 1-3 hours Newbie friendliness 86/100
guilhermerodz/input-otp#157 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
davidjerleke/embla-carousel#1414 ·
Maintainers usually reply within 4 days