Conversation
Icarus-B4
commented
Aug 4, 2025
Icarus-B4
commented
- Add DeleteSessionModal component for confirmation dialog
- Add delete button to each session item in SessionListView
- Implement deleteSessionById function using Electron API
- Add IPC handlers for session file deletion
- Add window transparency handlers for PR3 compatibility
- Update preload.ts with new API functions
- Fix TypeScript errors and imports
- Add DeleteSessionModal component for confirmation dialog - Add delete button to each session item in SessionListView - Implement deleteSessionById function using Electron API - Add IPC handlers for session file deletion - Add window transparency handlers for PR3 compatibility - Update preload.ts with new API functions - Fix TypeScript errors and imports
DOsinga
left a comment
DOsinga
left a comment
There was a problem hiding this comment.
I appreciate the enthusiasm but this is the third time you made me look at something that sill has traces of LLM work
| schedulingEngine: SchedulingEngine; | ||
| showQuitConfirmation: boolean; | ||
| enableWakelock: boolean; | ||
| windowOpacity?: number; |
There was a problem hiding this comment.
Sorry im new in PR.
Do i have to remove the comments?
Its not allowed to use AI?
...
There was a problem hiding this comment.
Hi @Icarus-B4 - again I like your enthusiasm and you are encourage to use AI!
however (and maybe we should post some guidelines here) it is your responsibility to make sure the PR the AI produces is actually ready for review. so you can't have comments in there that the AI left there (like "you could show a toast here") or in this case left overs from the previous PR (windowOpacity is not related to the current change). it would also behoove you to check out @zanesq link to the other implementation of this feature.
There was a problem hiding this comment.
Thanks for the clear guidance. Good point – I'll be more careful to clean up my PRs in the future and review existing implementations beforehand. I'll checked out @zanesq's link. its "Privat"
Any way i close the PR and let the professionals work.
2 participants
