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

Fix #8 for influx v2 #9

Closed
wants to merge 1 commit into from
Closed

Fix #8 for influx v2 #9

wants to merge 1 commit into from

Conversation

simonsickle
Copy link

Also update xml to unraid newer format

Also update xml to unraid newer format
@simonsickle
Copy link
Author

Fixes #8

@simonsickle
Copy link
Author

On second thought, maybe this should become influx2 template and we should create another to do influx1 for people that want to keep using that? Thoughts @atribe

@atribemc
Copy link

Yeah, that seems like a good idea since they are so different.

@simonsickle
Copy link
Author

I'll take that on tonight and update this PR. Would you like to maintain influx 1, 2 or both (template wise)? If you don't want 1 or don't want 2 I can put it in my repository too.

@atribemc
Copy link

i don't care. you can take it on

@simonsickle
Copy link
Author

I created 2 new templates (one for influx1 and one for influx2) - I have also asked Squidly what the preferred way is to handle depreciating this template. Depending on that conversation, this PR will get updated https://forums.unraid.net/topic/38619-docker-template-xml-schema/page/6/#comment-1155011

@simonsickle simonsickle closed this by deleting the head repository Sep 11, 2022
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.

None yet

2 participants