Skip to content

Refactor comment deletion and ensure all comment records are deleted - #1104

Merged
begedin merged 1 commit into
developfrom
1100-delete-comments-and-github-comments
Oct 23, 2017
Merged

Refactor comment deletion and ensure all comment records are deleted#1104
begedin merged 1 commit into
developfrom
1100-delete-comments-and-github-comments

Conversation

@joshsmith

@joshsmithjoshsmith commented Oct 22, 2017

Copy link
Copy Markdown
Contributor

What's in this PR?

  • Moves the comment deleter logic into the separate appropriate syncing responsibilities
  • Ensures both Comment and GithubComment records are deleted when a comment is deleted on GitHub

References

Fixes#1100

@joshsmith
joshsmithforce-pushed the 1100-delete-comments-and-github-comments branch from 5315772 to 95746bfCompareOctober 22, 2017 22:49
@begedin

Copy link
Copy Markdown
Contributor

@joshsmith I'm expecting to get conflicts in #1105 if I don't (at the very least in tests), so I'll merge this one myself.

@begedin
begedin merged commit f17e846 into developOct 23, 2017
@begedin
begedin deleted the 1100-delete-comments-and-github-comments branch October 23, 2017 07:24
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@joshsmith@begedin