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

Allow loading configuration from local network file #51

Open
rodrigoroma opened this issue Nov 8, 2019 · 4 comments
Open

Allow loading configuration from local network file #51

rodrigoroma opened this issue Nov 8, 2019 · 4 comments

Comments

@rodrigoroma
Copy link

Hi! First of all, thanks for this excellent project, it's exactly what I needed!

I'd like to host the config files in a shared drive. I tried to set the URL to file:///P:/xxxx/config.json (where P: is a network drive) but it's always "fetching..." and never finish. I tried with some variations of this URL, but all of them stays "fetching..." even if I can manually open them in a normal tab.

@KevinSheedy
Copy link
Owner

Hi @rodrigoroma
For now, I think you'll need to grant this permission to the plugin yourself by right-clicking the logo and clicking manage extensions.
Then look for the switch to Allow access to file URLs

image

In a future release, I may try request this permission at install time.

@rodrigoroma
Copy link
Author

Perfect! It couldn't had been easier!

This information could be in README file as it would give people the ideia that it's possible to host the config file in a network drive, and not necessarily in a web server =)

@KevinSheedy
Copy link
Owner

Good idea @rodrigoroma
If it's ok, I'll reopen this issue as a reminder to either add it to the README.md or request the permissions automatically during installation

@KevinSheedy KevinSheedy reopened this Nov 10, 2019
@rodrigoroma
Copy link
Author

I don't know if it's really necessary to request this permission during installation as people could be afraid of security/privacy issues. Maybe just explaining how to enable this permission in README or configuration interface would be enough.

What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants