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.

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

[GH-139] octopus icon randomly changes where it points to #164

Merged
merged 2 commits into from
Dec 11, 2019

Conversation

larkox
Copy link
Contributor

@larkox larkox commented Dec 2, 2019

Summary:
Fix "octopus icon randomly changes where it points to"

Issue link:
Fixes #139

@hanzei hanzei added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels Dec 2, 2019
Copy link
Member

@crspeller crspeller left a comment

Choose a reason for hiding this comment

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

Good fix!

@crspeller crspeller removed the 2: Dev Review Requires review by a core committer label Dec 2, 2019
@hanzei hanzei added this to the v0.13.0 milestone Dec 2, 2019
@DHaussermann
Copy link

This is performing as expected when no Enterprise Base URL is configure but, I'm unsure of how I can validate this when there is an Enterprise Base URL in place.

I can add a token URL to the config and hit refresh to see the icon pointing to it but I can't validate that it works while having a connected user. @crspeller does Mattermost have an enterprise URL I may not be aware of?

@crspeller
Copy link
Member

@jwilander Did we get something to test the GitHub enterprise integration?

@jwilander
Copy link
Member

I'm part of the GitHub Enterprise developer program which is how I tested this stuff in the past. I don't have that instance running anymore though unfortunately

It's pretty easy to get into the developer program and spin up an instance though. Might be a good idea to have a "permanent" instance for testing the plugin

Copy link

@DHaussermann DHaussermann left a comment

Choose a reason for hiding this comment

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

I have done the regression testing I can do without an Enterprise URL and confirmed with a token URL that the "Octocat" icon point to it correctly.

I suggest merging this. We could then post in #139 and see if someone can confirm the fix before we package a release for the plugin.

@DHaussermann DHaussermann added 4: Reviews Complete All reviewers have approved the pull request and removed 3: QA Review Requires review by a QA tester labels Dec 10, 2019
@hanzei hanzei closed this Dec 11, 2019
@hanzei hanzei reopened this Dec 11, 2019
@hanzei hanzei merged commit ebd8a34 into mattermost:master Dec 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

octopus icon randomly changes where it points to
6 participants