Skip to content
This repository was archived by the owner on Jan 22, 2019. It is now read-only.

fix: only show one code intel status indicator#251

Merged
chrismwendt merged 2 commits intomasterfrom
one-status-indicator
Oct 18, 2018
Merged

fix: only show one code intel status indicator#251
chrismwendt merged 2 commits intomasterfrom
one-status-indicator

Conversation

@chrismwendt
Copy link

This hides the legacy status indicator when Sourcegraph extensions are being used.

@felixfbecker
Copy link

Does this also remove the third one?

image

Copy link

@ijsnow ijsnow left a comment

Choose a reason for hiding this comment

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

This would be fine to merge but I'd rather not import browser storage from this file.

@chrismwendt
Copy link
Author

I'm not sure where the 3rd indicator icon is coming from, so after this is merged, you'll probably see 2.

@chrismwendt chrismwendt merged commit f944aa7 into master Oct 18, 2018
@chrismwendt chrismwendt deleted the one-status-indicator branch October 18, 2018 03:26
@felixfbecker
Copy link

Will check once released. Are you planning to cut a release?

@sourcegraph-bot
Copy link

🎉 This PR is included in version 1.17.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

4 participants