add hackatime project linking/unlinking/refresh #22
Conversation
![sentry[bot]](https://avatars.githubusercontent.com/u/1396951?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
This pull request adds Hackatime project linking, unlinking, and refresh functionality to the application. It enables users to associate their Hackatime time-tracking projects with application projects and automatically fetch/calculate time spent. The PR also automatically links Hackatime accounts during user registration.
Key changes:
- Added Hackatime account linking during user registration with automatic account lookup
- Implemented endpoints for linking/unlinking Hackatime projects to user projects and recalculating time
- Extended User model with Hackatime-related fields (hackatime_id, username, hackatime_last_fetched)
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 17 comments.
Show a summary per file
| File | Description |
|---|---|
| models/user.py | Added hackatime_id, username, and hackatime_last_fetched fields to User model; removed obsolete update_hackatime() method from UserProject |
| lib/hackatime.py | Added early return optimization for empty project filters to avoid unnecessary API calls |
| api/v1/users/main.py | Added hackatime fields to UserResponse; implemented /recalculate_time endpoint; added selectinload for projects relation |
| api/v1/projects/main.py | Implemented /hackatime endpoints for linking/unlinking projects; improved error messages with consistent capitalization; added authorization checks |
| api/v1/auth/main.py | Integrated automatic Hackatime account lookup during user registration flow |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 10 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 8 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
ImShyMike
requested review from
jleuth,
tailhaver and
thesleepyniko
and removed request for
jleuth and
thesleepyniko
|
lgtm! |
also adds hackatime account linking