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

Rewrite Companion docs #5

Merged
merged 21 commits into from
Jan 4, 2023
Merged

Rewrite Companion docs #5

merged 21 commits into from
Jan 4, 2023

Conversation

Murderlon
Copy link
Member

@Murderlon Murderlon commented Apr 21, 2022

  • Big rewrite with endless tiny tweaks
  • Add guide how to scale companion (sticky sessions, trade-offs)
  • Explain the redis options in detail, what do they enable?

https://transloadit.github.io/uppy.io/pr-preview/pr-5/docs/companion

@github-actions
Copy link
Contributor

github-actions bot commented Apr 21, 2022

PR Preview Action v1.2.0
Preview removed because the pull request was closed.
2023-01-04 13:38 UTC

docs/companion.md Outdated Show resolved Hide resolved
docs/companion.md Outdated Show resolved Hide resolved
Co-authored-by: Mikael Finstad <finstaden@gmail.com>
Copy link
Contributor

@mifi mifi left a comment

Choose a reason for hiding this comment

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

docs/companion.md Show resolved Hide resolved
docs/companion.md Outdated Show resolved Hide resolved
docs/companion.md Outdated Show resolved Hide resolved
docs/companion.md Show resolved Hide resolved
docs/companion.md Outdated Show resolved Hide resolved
mifi and others added 3 commits September 28, 2022 21:56
@mifi
Copy link
Contributor

mifi commented Nov 22, 2022

I have now added scaling docs from transloadit/uppy#4227 and improved a bit. Please let me know if there's anything else I should improve :)

@Murderlon
Copy link
Member Author

Let's first hammer out the feedback in the other PR, and then copy it here again :)

docs/companion.md Outdated Show resolved Hide resolved
also document some undocumented vars
docs/companion.md Outdated Show resolved Hide resolved
docs/companion.md Outdated Show resolved Hide resolved
@Murderlon Murderlon requested a review from mifi December 23, 2022 13:04
docs/companion.md Outdated Show resolved Hide resolved
@Murderlon Murderlon merged commit 59264b8 into main Jan 4, 2023
@Murderlon Murderlon deleted the companion-docs branch January 4, 2023 13:35
mifi added a commit that referenced this pull request Mar 28, 2023
from transloadit/uppy#4382

Also re-add `COMPANION_REDIS_EXPRESS_SESSION_PREFIX` - it was introduced in #5 (d890cb4) but somehow it's now gone. (could other things be gone as well in a botched merge?)

also add "This is a standalone-only option" to all standalone-only options. it was only added to one of them.
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.

3 participants