fedocal

#189 Make fedocal rely on fedocal-messages to check the structure of the messages sent
Merged by pingou. Opened by pingou.
fedora_message_schema  into  master

Download 189.patch
no initial comment

1 new commit added

  • Simplify how dicts are turned into fedora-messaging messages

pretty please pagure-ci rebuild

4 new commits added

  • Simplify how dicts are turned into fedora-messaging messages
  • Make fedocal rely on fedocal-messages to check the structure of the messages sent
  • Drop the flask10_only decorator and fix/adjust the tests
  • Port the multi-email validator to email_validator

5 new commits added

  • Only send a notification on the bus when a calendar has been cleared
  • Simplify how dicts are turned into fedora-messaging messages
  • Make fedocal rely on fedocal-messages to check the structure of the messages sent
  • Drop the flask10_only decorator and fix/adjust the tests
  • Port the multi-email validator to email_validator

Tests work locally, I like the simplifications of removing flask10_only decorator and using fedora_messages for validation, good to merge.

Thanks for the review (and catching all the issues you found!)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.