YunoHost-Apps / nextcloud_ynh

Nextcloud package for YunoHost
https://nextcloud.com/
GNU Affero General Public License v3.0
147 stars 63 forks source link

YNH Documentation about OnlyOffice could be reordered for clarity #624

Closed narFnarF closed 5 months ago

narFnarF commented 11 months ago

Describe the bug

The yunohost documentation on how to install Onlyoffice starts by describing how to install OnlyOffice using the Nextcloud app way, while warning that the performances are poor and lack ARM support, then explains that there's a better way, using the yunohost onlyoffice app.

Why not reorder the sections so that it explains the better way first? Wouldn't it be clearer?

Steps to reproduce

narFnarF commented 11 months ago

If someone point me in the right direction, I could make this change myself and do a pull request. Would I only need to change the Readme.md file? Is that what the yunohost admin interfaces uses to display at the bottom of https://example.com/yunohost/admin/#/apps/nextcloud/main ?

ericgaspar commented 11 months ago

This is the file to modify DESCRIPTION.md README.md is generated from it

narFnarF commented 10 months ago

I'm not super familiar with Github's pull-requests. I hope I did it properly?

lapineige commented 10 months ago

634 sounds good, I'm not sure about #635 (it not targeting testing, hence there is a lot of patches from the testing branch that you forked). Can you change the target to be testing ?

narFnarF commented 10 months ago

I think I fixed it?

lapineige commented 10 months ago

Perfect :)

lapineige commented 5 months ago

Should we close ?

narFnarF commented 5 months ago

yes