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

make Github icon url configurable #57

Closed
wants to merge 1 commit into from

Conversation

zetaab
Copy link
Contributor

@zetaab zetaab commented Mar 5, 2019

I installed 0.8 version today our mattermost, and now the problem is that Icon is not visible at all. This is because icon url is hardcoded to code, and our mattermost server cannot access to internet.

That is why I suggest that we make this url configurable

@zetaab
Copy link
Contributor Author

zetaab commented Mar 5, 2019

I have no idea why that test fails

@lieut-data
Copy link
Member

Is there a way we could have the plugin host the image itself? Or for the client-side code to embed the image into the webpack bundle directly?

Presumably this doesn't need to actually be "configurable", but just accessible without an internet connection. Or, @zetaab, were you hoping to allow changing it from the default logo?

@zetaab
Copy link
Contributor Author

zetaab commented Mar 5, 2019

Yes I hope that i could change the url of the logo. This code works fine for me. I am now hosting that same image inside our internal network and it works fine. This feature also allows changing the bot icon if someone wants customize it

@lieut-data lieut-data added 1: PM Review Requires review by a product manager 2: Dev Review Requires review by a core committer labels Mar 5, 2019
@jwilander
Copy link
Member

I do have a PR here for self-hosting the image from the plugin #50

However, I'm not opposed to making it configurable

Copy link
Contributor

@jasonblais jasonblais left a comment

Choose a reason for hiding this comment

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

I suspect majority of admins won't care about configuring the URL (they just want it to "work"), so I'd prefer the self-hosting option over making it configurable. Which #50 would cover, sounds like.

@crspeller
Copy link
Member

Agree with @jasonblais #50 is the solution to this. Thanks for the contribution, but going to close this.

@crspeller crspeller closed this Mar 6, 2019
IndushaS pushed a commit to IndushaS/mattermost-plugin-github that referenced this pull request Feb 8, 2021
The cloud server now support single and multitenant PostgreSQL
databases. This change adds both of these options and sets the
default database to multitenant Postgres for new installations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1: PM Review Requires review by a product manager 2: Dev Review Requires review by a core committer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants