Move DesktopNotifier.app_icon to DesktopNotifierBackend.app_icon

[PhrozenByte]

This truly freezes Notification instances

Resource.as_name() is necessary to make mypy happy, otherwise it always assumes that Resource.name might be None

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 87.17949% with 5 lines in your changes missing coverage. Please review.

Project coverage is 84.89%. Comparing base (26c6af2) to head (13638cd).
Report is 14 commits behind head on main.

Files with missing lines	Patch %	Lines
src/desktop_notifier/main.py	71.42%	2 Missing ⚠️
src/desktop_notifier/sync.py	66.66%	2 Missing ⚠️
src/desktop_notifier/common.py	75.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #202      +/-   ##
==========================================
- Coverage   85.13%   84.89%   -0.25%     
==========================================
  Files          10       10              
  Lines         895      920      +25     
==========================================
+ Hits          762      781      +19     
- Misses        133      139       +6     

Flag	Coverage Δ
pytest	84.89% <87.17%> (-0.25%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[samschott]

Thanks! The current icon handling was indeed a bit of a hack.