ComputerScienceHouse / ComputerScienceHouse/conditional

GET request to mark HM as Attended

Open
#145 0 comments 0 reactions 0 assignees View on GitHub
good first issue tech debt
Dominant language
Python
Stars
12
Forks
39
Avg merge
1d 13h
Merged PRs (30d)
1

Description

For some inexplicable reason I made ```@attendance_bp.route('/attendance/alter/hm//', methods=['GET'])``` a `GET` request. No idea why, it totally breaks the RESTful standards.

Nonetheless, it should be fixed.

Contributor guide

No contributing guide indexed for this repository

Research direction

Find the Flask route matching /attendance/alter/hm// and inspect how the attendance action is invoked. Replace the current GET-only behavior with the appropriate method for changing attendance state, then verify that marking an HM attended still works and that GET is no longer used for the change.

Written by the indexing model from the issue text.

Assessment

Tech stack
flask, python
Domain
api, backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.