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

quickfixes für issue #77 #78

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

aruhri
Copy link

@aruhri aruhri commented Oct 19, 2021

This is a quick and dirty fix wich made my installation for docker run.
closes #77

@iDebugAll
Copy link
Owner

Hi @aruhri. Thank you for the PR.
Dockerfile fix looks good. I'll test in my environment shortly.
Please revert the changes to the template files regarding older jquery versions. I have left those links intentionally for backward compatibility with older NetBox versions.

@aruhri
Copy link
Author

aruhri commented Oct 20, 2021

I think it is not necessary, because the plugin delivers the jquery itself. It does not use jquery installed in netbox.

@RobinBeismann
Copy link

@iDebugAll , can we get this merged? At the moment the docker build in the main branch does not work but this PR fixes it.

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

Successfully merging this pull request may close these issues.

Serveral issues with access-rights prevent from displaying the content in docker-image
3 participants