Some cleanup around champ updates

[tchak]

No description provided.

[sentry-io]

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: app/controllers/attachments_controller.rb

Function	Unhandled Issue
destroy	[**NoMethodError: undefined method logo' for nil (NoMethodError)**](https://demarches-simplifiees.sentry.io/issues/6099355306/?referrer=github-open-pr-bot) AttachmentsCont... <br> Event Count:` 1

---

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

[codecov]

Codecov Report

Attention: Patch coverage is 73.33333% with 4 lines in your changes missing coverage. Please review.

Project coverage is 79.96%. Comparing base (fc0f39b) to head (6b8bc4b).

Files with missing lines	Patch %	Lines
...ponents/editable_champ/editable_champ_component.rb	0.00%	2 Missing ⚠️
app/controllers/attachments_controller.rb	83.33%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #11110      +/-   ##
==========================================
- Coverage   84.45%   79.96%   -4.49%     
==========================================
  Files        1187     1187              
  Lines       26238    26515     +277     
  Branches     4941     4851      -90     
==========================================
- Hits        22158    21203     -955     
- Misses       4080     5312    +1232     

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