Skip to content

feat(settings): replace OnlyOffice with Euro-Office in office suite s…#59933

Open
moodyjmz wants to merge 3 commits into
masterfrom
feat/euro-office-switcher
Open

feat(settings): replace OnlyOffice with Euro-Office in office suite s…#59933
moodyjmz wants to merge 3 commits into
masterfrom
feat/euro-office-switcher

Conversation

@moodyjmz
Copy link
Copy Markdown

@moodyjmz moodyjmz commented Apr 27, 2026

…witcher

  • Resolves: #

Summary

Changes the OnlyOffice entry for Euro-Office
image

image

How to setup

Testing in the Euro-Office dev environment
Create a temporary Dockerfile.nextcloud in your local checkout of the server branch root do not commit:

FROM nextcloud:latest
COPY dist/settings-apps-view-4529.js /usr/src/nextcloud/dist/settings-apps-view-4529.js
COPY dist/settings-apps-view-4529.js.map /usr/src/nextcloud/dist/settings-apps-view-4529.js.map

Then in develop/docker-compose.yml, temporarily replace:
image: nextcloud:latest
with:
build:
context: /path/to/server
dockerfile: Dockerfile.nextcloud
Run docker compose build nextcloud && docker compose up -d --force-recreate nextcloud, then open http://localhost:8081/settings/apps/office.

TODO

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@moodyjmz moodyjmz self-assigned this Apr 27, 2026
@moodyjmz moodyjmz requested a review from a team as a code owner April 27, 2026 13:17
@moodyjmz moodyjmz requested review from nfebe, susnux and szaimen and removed request for a team April 27, 2026 13:17
@moodyjmz moodyjmz marked this pull request as draft April 27, 2026 13:17
Comment thread apps/settings/src/constants/OfficeSuites.js Outdated
@susnux susnux requested review from juliusknorr and removed request for susnux April 27, 2026 15:42
@karlitschek
Copy link
Copy Markdown
Member

@jospoortvliet @jancborchardt what do you think?

Copy link
Copy Markdown
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Nice! :) I think:

  • "Nextcloud Office" should be the left entry, default selected, but that would be the one powered by Euro-Office
  • "Powered by Euro-Office" could be the first bullet point beneath that
  • "Collabora Office" should be the right entry

That uses the same logic as before: The default is called "Nextcloud Office", whatever it’s powered by. The alternative uses its brand name.

@moodyjmz
Copy link
Copy Markdown
Author

moodyjmz commented May 8, 2026

Nice! :) I think:

  • "Nextcloud Office" should be the left entry, default selected, but that would be the one powered by Euro-Office
  • "Powered by Euro-Office" could be the first bullet point beneath that
  • "Collabora Office" should be the right entry

That uses the same logic as before: The default is called "Nextcloud Office", whatever it’s powered by. The alternative uses its brand name.

I have made these changes @elzody can you review this?

@moodyjmz moodyjmz requested a review from elzody May 8, 2026 12:14
@jancborchardt jancborchardt moved this to 🏗️ At engineering in 🖍 Design team May 27, 2026
@juliusknorr juliusknorr moved this from 📄 To do to 🏗️ In progress in 📄 Euro-Office team May 28, 2026
@moodyjmz moodyjmz force-pushed the feat/euro-office-switcher branch from 0ff1beb to 5b1a67a Compare May 28, 2026 10:08
@juliusknorr juliusknorr marked this pull request as ready for review May 28, 2026 10:11
moodyjmz and others added 2 commits May 28, 2026 12:12
…witcher

- Replace onlyoffice/onlyoffice entry with collabora-office/richdocuments
- Change nextcloud-office appId from richdocuments to eurooffice
- Update features and learnMoreUrl for Euro-Office branding

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: James Manuel <moodyjmz@users.noreply.github.com>
…alled

Falls back to the isPrimary suite so the UI shows a pre-selection rather
than nothing, without triggering the watcher (watch() is lazy by default).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: James Manuel <moodyjmz@users.noreply.github.com>
@moodyjmz moodyjmz force-pushed the feat/euro-office-switcher branch from 5b1a67a to 50ada12 Compare May 28, 2026 10:12
@juliusknorr
Copy link
Copy Markdown
Member

/backport! to stable34

@juliusknorr
Copy link
Copy Markdown
Member

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🏗️ At engineering

Development

Successfully merging this pull request may close these issues.

8 participants