Fix user content deletion

[nilsding]

No description provided.

[codecov]

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (b8fd519) 95.20% compared to head (c4da510) 95.20%.
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1397   +/-   ##
=======================================
  Coverage   95.20%   95.20%           
=======================================
  Files         168      168           
  Lines        2629     2629           
=======================================
  Hits         2503     2503           
  Misses        126      126           

Files	Coverage Δ
app/models/inbox.rb	92.85% <66.66%> (ø)
app/models/notification.rb	92.85% <66.66%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pixeldesu]

Works, but it now surfaced that the same is also the case for Notification. Procedure of fixing should be exactly the same as with Inbox already.

[ActiveJob] [ActiveRecord::DestroyAssociationAsyncJob] [ba2aaeeb-5f20-4015-9dda-9ef237a55270] Error performing ActiveRecord::DestroyAssociationAsyncJob (Job ID: ba2aaeeb-5f20-4015-9dda-9ef237a55270) from Sidekiq(default) in 102.54ms: NoMethodError (undefined method `touch' for nil:NilClass

    recipient.touch(:notifications_updated_at)
             ^^^^^^):
/app/app/models/notification.rb:16:in `block in <class:Notification>'

https://github.com/Retrospring/retrospring/blob/main/app/models/notification.rb#L16

[nilsding]

nice catch, patched this one too

[sentry-io]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ ActiveRecord::Deadlocked: PG::TRDeadlockDetected: ERROR: deadlock detected (ActiveRecord::Deadlocked) Sidekiq/ActiveRecord::DestroyAssociationAsyncJob View Issue

_{Did you find this useful? React with a 👍 or 👎}

[nilsding]

ja well…