Catch and handle errors of FCM

[MizukiTemma]

Short description

This PR add an error handling for invalid FCM URL.

update
This takes care of timeout exception too. I've totally forgotten that those two issues are related each other.

Proposed changes

• Catch errors and show an error message to users
• The error message is left universal (not specific to any error type) because many types are possible.

Side effects

• The rendering after failing of a push notification is partially broken.
• A question regarding this rendering: don't we want to redirect not no the create view but to the edit view becuase the push notification is saved in this case?
• I'll suggest to fix the rendering in a separate PR/issue.

Resolved issues

Fixes: #2025 , #2790

---

Pull Request Review Guidelines

[codeclimate]

Code Climate has analyzed commit b19104e and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 15.3% (50% is the threshold).

This pull request will bring the total coverage in the repository to 82.2% (0.0% change).

View more on Code Climate.

[david-venhoff]

Thanks, this fixes both linked issues.
I think, as part of the clean code initiative, we should avoid introducing new pylint: disable= comments. Maybe one option would be to handle the errors directly at self.send_pn

[MizukiTemma]

@david-venhoff
That's a nice idea 😸 Applied!

[david-venhoff]

Nice :)

[ulliholtgrave]

Thanks for adding this extra handling 👍