-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Adjust recommended apps page #34084
Adjust recommended apps page #34084
Conversation
wait, now I realize that we only wanted to remove collabora code from the list, assuming that it would be installed anyway. but now I renamed it from the code, so it won't be installed... I'll bring it back on the backend only... |
Removed collabora code. Added description for richdocuments as the name will be "Nextcloud Office" pulled from the app store. Signed-off-by: Vincent Petry <[email protected]>
8c7f347
to
71eab82
Compare
okay, I pushed again, now, as per requirements: richdocumentscode doesn't appear in the visible list but will still be installed |
and, I added a name override for Collabora Office so that it always appears as "Nextcloud Office" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but didn't test
BTW @PVince81 didn't we also put recognize into recommended apps? |
Removed collabora code.
Added description for richdocuments as the name will be "Nextcloud Office" pulled from the app store.
Note: currently it still says "Collabora Online" and the text will change as soon as the app store name changes to "Nextcloud Office" as well in the next app release: https://apps.nextcloud.com/apps/richdocuments