-
Notifications
You must be signed in to change notification settings - Fork 86
Closed
Description
Problem
GithubRepo needs new columns like:
sync_stateunsynced(default)fetching_pull_requestserrored_fetching_pull_requestssyncing_pull_requestserrored_syncing_pull_requestsfetching_issueserrored_fetching_issuessyncing_issueserrored_syncing_issuesfetching_commentserrored_fetching_commentssyncing_commentserrored_syncing_commentsreceiving_webhooks(final state, indicating it’s now just listening)disconnected?
syncing_pull_requests_countsyncing_issues_countsyncing_comments_count
For the sync_state, we'll want to add changeset validation on inclusion of those columns.
The question around the disconnected state surrounds whether we want to outright remove a GithubRepo from a project, or simply set it to disconnected (so the user can resume syncing without losing the existing data).
This is probably a bigger question that subsumes this issue and largely lies outside of it. My vote is to not add it for now, but to potentially have this discussion in another issue. Thoughts @begedin and others?