Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

Conversation

@orta
Copy link

@orta orta commented Oct 7, 2017

This hides the extension viewport by default and then adds the viewport when it's been confirmed that you're in a GitHub repo.

@chrmarti chrmarti added this to the October 2017 milestone Oct 7, 2017
Copy link
Contributor

@chrmarti chrmarti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! Please see my review comments.

"id": "githubIssuesPrs",
"name": "GitHub Issues and PRs"
"name": "GitHub Issues and PRs",
"when": "is-running-a-github-repo"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This appears to prevent the extension from activating with the "onView:githubIssuesPrs" event (see further up in this file). Activating on "*" should be good enough for now.

}

private async fetchChildren(element?: TreeItem): Promise<TreeItem[]> {
const remotes = await this.getGitHubRemotes();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We seem to only get here when the view is already activated.

@chrmarti chrmarti removed this from the October 2017 milestone Oct 30, 2017
Base automatically changed from master to main March 3, 2021 06:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants