FEATURE: per-topic unsubscribe option in emails#2
Conversation
|
cursor review |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Bugbot free trial expires on August 11, 2025
Learn more in the Cursor dashboard.
| tu.save! | ||
|
|
||
| perform_show_response | ||
| end |
There was a problem hiding this comment.
Bug: Unsubscribe Fails with Nil User or TopicUser
The unsubscribe method in TopicsController can raise a NoMethodError. This occurs if current_user is nil (e.g., anonymous access) when attempting to access current_user.id. It also occurs if TopicUser.find_by returns nil (meaning no existing record for the user/topic combination), leading to attempts to access tu.notification_level or call tu.save! on a nil object.
Locations (1)
|
This pull request has been automatically marked as stale because it has been open for 60 days with no activity. To keep it open, remove the stale tag, push code, or add a comment. Otherwise, it will be closed in 14 days. |
3 participants
Test 2