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

cloudflare access documentation #10755

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

matejkramny
Copy link
Contributor

complements PR #10588

adds documentation section under Guides for cloudflare access, detailing how to set up cloudflare to work with Immich and how to set up headers in the app

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jul 2, 2024
Copy link
Member

@danieldietzler danieldietzler left a comment

Choose a reason for hiding this comment

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

As this is prone to be outdated, I'd prefer this to be a community guide. (like setting up cloudflare tunnel)

Copy link

github-actions bot commented Jul 2, 2024

📖 Documentation deployed to pr-10755.preview.immich.app

@mmomjian
Copy link
Contributor

mmomjian commented Jul 2, 2024

I think we already have this? Access Immich with custom domain - https://github.com/ppr88/immich-guides/blob/main/open-immich-custom-domain.md

I'd recommend any changes as a PR to that project

@matejkramny
Copy link
Contributor Author

Hi, I've added the content to https://github.com/matejkramny/immich-guides/tree/main
It might not get through, as there's no activity from the guide creator. ppr88/immich-guides#1

I'll give the original creator some time to see the PR, if not, would you be ok replacing the community guide with my fork?

@danieldietzler
Copy link
Member

danieldietzler commented Jul 8, 2024

I'll give the original creator some time to see the PR, if not, would you be ok replacing the community guide with my fork?

Absolutely!

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

Successfully merging this pull request may close these issues.

None yet

3 participants