Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not send RabbitMQ event when updating scheduled article #3097

Merged
merged 1 commit into from
Jan 24, 2023

Conversation

nickskalkin
Copy link
Contributor

Currently each time I save a scheduled article - the rabbitmq event is being published.

@nickskalkin nickskalkin self-assigned this Jan 24, 2023
@codecov
Copy link

codecov bot commented Jan 24, 2023

Codecov Report

Merging #3097 (37b9bcb) into main (83ea71b) will increase coverage by 0.0%.
The diff coverage is 100.0%.

❗ Current head 37b9bcb differs from pull request most recent head a6fd43e. Consider uploading reports for the commit a6fd43e to get more accurate results

@@          Coverage Diff          @@
##            main   #3097   +/-   ##
=====================================
  Coverage   83.0%   83.0%           
=====================================
  Files        197     197           
  Lines       5410    5413    +3     
  Branches     989     989           
=====================================
+ Hits        4495    4498    +3     
  Misses       662     662           
  Partials     253     253           

@nickskalkin nickskalkin force-pushed the nickskalkin/do-not-send-rabbit-message-draft-save branch from 37b9bcb to a6fd43e Compare January 24, 2023 09:22
@nickskalkin nickskalkin merged commit 5273ab4 into main Jan 24, 2023
@nickskalkin nickskalkin deleted the nickskalkin/do-not-send-rabbit-message-draft-save branch January 24, 2023 09:30
@artsy-peril artsy-peril bot mentioned this pull request Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant