Skip to content
This repository has been archived by the owner on Oct 21, 2021. It is now read-only.

Plugin says it's disabled, and has access to the site #156

Open
fstani opened this issue Jun 16, 2020 · 3 comments · May be fixed by #174
Open

Plugin says it's disabled, and has access to the site #156

fstani opened this issue Jun 16, 2020 · 3 comments · May be fixed by #174

Comments

@fstani
Copy link

fstani commented Jun 16, 2020

Plugin on Chrome and Firefox will say it's disabled, IDK how to actually enable it.

Mattermost Version: 5.21.0
Database Schema Version: 5.21.0

Chrome Version 83.0.4103.97 (Official Build) (64-bit):
image

My mattermost is from the gitlab onmibus, idk if that has anything to do with this

I don't see any errors on chrome dev console, or broken network requests, my server runs on our own custom domain as well.

Any help is greatly appreciated

@ivangabriele
Copy link
Owner

Hi @fstani,

It's disabled if it doesn't detect the url as a Mattermost one. What's the url of your Mattermost?

@fstani
Copy link
Author

fstani commented Jun 17, 2020

Hey @ivangabriele we run our matterrmost on our own domain, is there a setting to customize the URL? If not perhaps that would be a good addition to the plug-in code

@eyalzek
Copy link

eyalzek commented Sep 23, 2020

Same for me on firefox, we use a custom domain that definitely isn't matched by the regex in

!/^https:\/\/([^/]+\.mattermost|mattermost\.[^/]+|[^/]+\.mattermost\.[^/]+)/.test(url)

eyalzek added a commit to eyalzek/mattermost-uno that referenced this issue Sep 23, 2020
the regex
`/^https:\/\/([^/]+\.mattermost|mattermost\.[^/]+|[^/]+\.mattermost\.[^/]+)/`
doesn't work when one is using a custom url for their mattermost
instance. This PR adds an options page where a user can configure their
custom mattermost url.
Fixes ivangabriele#156
@eyalzek eyalzek linked a pull request Sep 23, 2020 that will close this issue
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 a pull request may close this issue.

3 participants