Skip to content

Added the possibility to preserve the sorting state on the dashboard#885

Open
frankhildebrandt wants to merge 1 commit intoRotherOSS:rel-10_1from
Freicon:dashboard-sort-order-preservation
Open

Added the possibility to preserve the sorting state on the dashboard#885
frankhildebrandt wants to merge 1 commit intoRotherOSS:rel-10_1from
Freicon:dashboard-sort-order-preservation

Conversation

@frankhildebrandt
Copy link
Contributor

Currently the dashboard looses its ordering configuration on every reload. As many other views store their order configuration in the user preferences I tried to adapt this behavior.

I'm not sure if this behavior needs to by Enabled/Disabled by sysconfig, or if the user-parameter keys violate some rules.

@frankhildebrandt frankhildebrandt force-pushed the dashboard-sort-order-preservation branch from 35a6766 to d575dca Compare March 25, 2021 10:33
@frankhildebrandt frankhildebrandt force-pushed the dashboard-sort-order-preservation branch from d575dca to 1a931be Compare March 25, 2021 10:46
@svenoe
Copy link
Contributor

svenoe commented Apr 14, 2021

Hi Frank,

sorry for the delay, I didn't find much time to look at it the last weeks, and thank you for your contribution.

In general I guess we could add this, although the other views as far as I have seen store only the filters, not the ordering and sorting. The Dashboard differs a little anyways, though. However, we would definitely need a SysConfig-Option for this. I can already see the complaints about overlooked tickets, because people are not aware that they changed their SortBy-option away from age the day before. (This also happens with filters regularly, but there at least you have to do slightly more than one little click.)

Secondly, there is no explicit naming convention, yet, but at least the camel case with the starting capital letter is used in OTRS/OTOBO everywhere, and looking at the other preferences, your suggestion looks quite different. I would prefer something like "UserDashboardSort-$Name" or similar.

Best regards, Sven

@frankhildebrandt
Copy link
Contributor Author

Sorry for my delay. This week I will update my request to match your requirements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments