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.

Sign up for GitHub

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

Jump to bottom

Collection for Dockge Docker Container. #1185

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

Collection for Dockge Docker Container. #1185

wants to merge 15 commits into from

Conversation

LearningSpot
Copy link

I have created this collection to protect Dockge docker container. I'm using this and working fine. Hope others can use this too.

@buixor
Copy link
Contributor

buixor commented Dec 5, 2024

thx for contrib, having a look :)

@LearningSpot
Resolve issues where in android app IP gets bans in the background fo…
e33bfcc 
…r connecting to nextcloud app.

I have faced this issue and used this as custom whitelist on 28 September 2024 and then no issues till now. not sure if the issue still there or have been fixed in any recent releases but I didn't find anything regarding this issue fix yet.
@LearningSpot
Copy link
Author

thx for contrib, having a look :)

Thanks.

@LearningSpot
Copy link
Author

Hi, I have added another collection for Open WebUI and a Fix for nextcloud whitelist #1102. I wanted to open new pull request but I'm still new with GitHub and somehow it pulled in this request. Sorry about that. Hope this wont complicate things for you to review. Anything please let me know. Thanks again

@LaurenceJJones
Copy link
Contributor

Hi, I have added another collection for Open WebUI and a Fix for nextcloud whitelist #1102. I wanted to open new pull request but I'm still new with GitHub and somehow it pulled in this request. Sorry about that. Hope this wont complicate things for you to review. Anything please let me know. Thanks again

All good! the better approach is to never work on master or main and simply create a branch to work on, however, this can cause issue if you already modified master or main since it will pull across your changes unless you wipe your changes.

But dont worry we love a 2 in 1 offer and can handle this in a single PR 👍🏻

@LearningSpot
Copy link
Author

I'm really sorry about this and thanks for your understanding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@LearningSpot @buixor @LaurenceJJones