external icon indicating copy to clipboard operation
external copied to clipboard

Add an option to reorder the links

Open nickvergessen opened this issue 6 years ago • 7 comments

Extracted from https://github.com/nextcloud/external/issues/69 by @Dubidubiduu

Then I would appreciate a way to change the order of my external sites which is not possible. But there is an App to change the order of menu items ("App Order"), so this is not a problem, but a small hint in the description of this app would be a nice thing.

nickvergessen avatar Jan 02 '18 12:01 nickvergessen

Actually only a few external links are shown in the "app order"-app. It's just those, who are free for all (no group limitation). But since also the apps with group limitations aren't shown in the app-order settings it seems the problem relies completely to that app.

Dubidubiduu avatar Apr 25 '20 10:04 Dubidubiduu

@Dubidubiduu Well the app-order app does work, as long as your external sites are all in the app menu. In that case it's fine, and I've actually been doing that for a while now. My current issue is that I've placed some external sites in the settings drop down because they are more admin related, and I've used the external sites group restrictions to block the sites from being shown to normal users. In that case there is no way at all to reorder them without deleting and adding them again, as far as I can tell.

Luticus avatar Apr 25 '20 16:04 Luticus

@Luticus Yeah but for this I added an issue in the app-order-app https://github.com/juliushaertl/apporder/issues/87

Dubidubiduu avatar Apr 25 '20 21:04 Dubidubiduu

Defining an app order based on groups or and admin could set a specific app order on a user would be great, that way a user did not get an access denied if it has not permissions to a specific app. Another way is app order jump to the next permitted app in the specified order. that would be great !

nunomdmgoncalves avatar May 06 '20 15:05 nunomdmgoncalves

@Dubidubiduu I think there's some confusion here as I don't see how that app order issue will address what I'm a talking about.

Basically if the app/icon/menu item is in the apps list: apporderbar Then I agree, the app order app helps a lot.

However if the items are in this menu: appordersettingmenu Then the app order app has no bearing on their position and they cannot easily be rearranged.

My issue isn't really about what different users see. If I want to hide a link from a user, I can already do that with the external sites app. You can already set a default app order, and if users want to overwrite that with their own, then that's fine. We shouldn't be forcing a users interface to look a certain way with the exceptions of setting default positions for things and hiding stuff they aren't supposed to see.

Luticus avatar May 06 '20 16:05 Luticus

Hello there, Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the reviewing process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, we would appreciate your feedback on your experience with our community management team.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

github-actions[bot] avatar Jun 22 '23 12:06 github-actions[bot]

For anyone curious, this can be achieved through some database wrangling. The following is how I did it with Postgres:

Enter the postgres command line, find your DB, and use it:

sudo -i -u postgres
psql
\l
\c {{ YOUR NEXTCLOUD DATABASE NAME }}

Grab the config:

select configvalue from oc_appconfig WHERE appid = 'external' AND configkey = 'sites';

Pipe this into your favourite JSON viewer / expander / indent tool.

Move the order of them around, and then re-number the IDs (both in the record and at the start of it).

Remove all new lines from the updated JSON to put it back in the same format as you found it.

Apply the updated config:

UPDATE oc_appconfig SET  configvalue = {{ YOUR MINIFIED JSON }} WHERE appid = 'external' AND configkey = 'sites';

nroach44 avatar Jul 19 '23 10:07 nroach44