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

Discount bandit #719

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

Conversation

Findarato
Copy link
Contributor

Added support for discount-bandit

linuxserver.io


  • I have read the contributing guideline and understand that I have made the correct modifications

Description

Benefits of this PR and context

How Has This Been Tested?

I am using this in my homelab right now.

Source / References

@Findarato Findarato mentioned this pull request Nov 25, 2024
1 task
@Findarato
Copy link
Contributor Author

I do not think the error is with my pull.

Copy link
Contributor

@pagdot pagdot left a comment

Choose a reason for hiding this comment

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

I think you missed a few cosmetic things :)

discountbandit.subdomain.conf.sample Outdated Show resolved Hide resolved
discountbandit.subdomain.conf.sample Outdated Show resolved Hide resolved
discountbandit.subfolder.conf.sample Outdated Show resolved Hide resolved
discountbandit.subfolder.conf.sample Outdated Show resolved Hide resolved
@Findarato
Copy link
Contributor Author

The Check Samples is failing on error pages not related to the pull

@aptalca
Copy link
Member

aptalca commented Dec 19, 2024

You might want to do a proper rebase and force push. A brand new PR passes the conf validity check: #723

Also, please remove the http2 directives

@aptalca
Copy link
Member

aptalca commented Dec 19, 2024

hold on, I take it back

You double included the proxy.conf in the subfolder.conf.sample

@Findarato Findarato requested a review from drizuid December 26, 2024 16:23
Copy link
Member

@drizuid drizuid left a comment

Choose a reason for hiding this comment

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

@Findarato
Copy link
Contributor Author

I am mostly sure I did the rebase correctly. Crazy how many things change with this repo while I was fixing the mistakes.

@Findarato Findarato requested a review from drizuid January 1, 2025 22:33
Copy link
Contributor Author

@Findarato Findarato left a comment

Choose a reason for hiding this comment

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

Done

@Findarato
Copy link
Contributor Author

#719 (comment)

I have completed it with e2b2c6c

@YouveGotMeowxy

This comment was marked as off-topic.

Copy link
Member

Choose a reason for hiding this comment

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

why is calibre-web in this pr?

@YouveGotMeowxy

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: PRs Ready For Team Review
Development

Successfully merging this pull request may close these issues.

8 participants