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

Feature: Add Collabora as a subchart #611

Closed
3 tasks
jessebot opened this issue Jul 26, 2024 · 0 comments · Fixed by #623
Closed
3 tasks

Feature: Add Collabora as a subchart #611

jessebot opened this issue Jul 26, 2024 · 0 comments · Fixed by #623
Labels
enhancement New feature or request

Comments

@jessebot
Copy link
Collaborator

Description of the change

Let's add support for Collabora as a subchart. Here's their helm chart and here's their docs.

Benefits

Allow users to have a document server deployed in this stack.

Possible drawbacks

  • Another thing to support.
  • Unknown: Not sure how much we can facilitate actually configuring the connection between Nextcloud and Collabora 🤷

Happy to discuss anything further!

Tasks

These are the tasks that would need to be done for this issue to be considered complete:

  • add tests in ci and/or helm unit-tests
  • add Collabora as a configurable sub-chart via parameters in values.yaml
  • install the app is there is one?

We can add more tasks if needed.
@jessebot jessebot added the enhancement New feature or request label Jul 26, 2024
@jessebot jessebot mentioned this issue Jul 26, 2024
Closed
@kriegalex kriegalex mentioned this issue Aug 23, 2024
Merged
4 tasks
jessebot added a commit that referenced this issue Dec 15, 2024
@kriegalex @jessebot
Feature: Add Collabora as a subchart #611 (#623)
10cae24 
* Add collabora subchart

Signed-off-by: kriegalex <[email protected]>

* Add collabora config to README

Signed-off-by: kriegalex <[email protected]>

* Update lint-test.yaml - add collabora-online to helm repos for chart testing

Signed-off-by: Jesse Hitch <[email protected]>

* Update charts/nextcloud/values.yaml - remove trailing spaces

Signed-off-by: Jesse Hitch <[email protected]>

---------

Signed-off-by: kriegalex <[email protected]>
Signed-off-by: Jesse Hitch <[email protected]>
Co-authored-by: Jesse Hitch <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Feature: Add Collabora as a subchart #611 kriegalex/nextcloud-helm
1 participant
@jessebot