b7c24e1d69
After #19666, we send notifications asynchronously. However, this has dramatically delayed the different post-send warnings we send. This change moves all the post-send warnings logic from `Chat::ChatNotifier` into a separate service, allowing us to run this part synchronously, displaying the warnings almost instantly without doing all the heavy lifting for sending notifications. It also simplifies the notifier, making another refactor we planned simpler. Finally, this removes the need to run jobs on the system specs that test the post-warning logic, hopefully making them faster. |
||
|---|---|---|
| .. | ||
| chat_channel_archive_service_spec.rb | ||
| chat_channel_fetcher_spec.rb | ||
| chat_channel_hashtag_data_source_spec.rb | ||
| chat_channel_membership_manager_spec.rb | ||
| chat_message_bookmarkable_spec.rb | ||
| chat_message_processor_spec.rb | ||
| chat_message_reactor_spec.rb | ||
| chat_notifier_spec.rb | ||
| chat_review_queue_spec.rb | ||
| chat_statistics_spec.rb | ||
| chat_transcript_service_spec.rb | ||
| direct_message_channel_creator_spec.rb | ||
| duplicate_message_validator_spec.rb | ||
| endpoint_spec.rb | ||
| guardian_extensions_spec.rb | ||
| message_mention_warnings_spec.rb | ||
| message_mover_spec.rb | ||
| post_notification_handler_spec.rb | ||
| slack_compatibility_spec.rb | ||
| steps_inspector_spec.rb | ||