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.

Sign up for GitHub

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

Jump to bottom

[CORL-1240] Check existing data before handling subscriptions #3077

Merged
merged 3 commits into from Aug 5, 2020
Merged

[CORL-1240] Check existing data before handling subscriptions #3077

merged 3 commits into from Aug 5, 2020

Conversation

cvle
Copy link
Member

@cvle cvle commented Aug 5, 2020

What does this PR do?

  • Make sure incoming subscription data is not already present and thus not need to be processed.

How do I test this PR?

  • Check live updates (Comment Created, Comment Released, Reply Created, Unanswered Created, Unanswered Released, Comment moderated, Single Comment moderated)

@cvle cvle requested a review from wyattjoh August 5, 2020 18:31
@wyattjoh wyattjoh added the 🚀 merge it! Pull requests that should be merged after status checks pass with a review label Aug 5, 2020
@wyattjoh wyattjoh added this to the v6.3.2 milestone Aug 5, 2020
@kodiakhq kodiakhq bot merged commit 6a8d6b0 into master Aug 5, 2020
@kodiakhq kodiakhq bot deleted the bug/CORL-1240 branch August 5, 2020 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wyattjoh wyattjoh wyattjoh approved these changes

Assignees
No one assigned
Labels
🚀 merge it! Pull requests that should be merged after status checks pass with a review
Projects
None yet
Milestone
v6.3.2
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@cvle @wyattjoh