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

ReGaHss: add SortByName() method to enum UserSharedObjects() #411

Closed
jp112sdl opened this Issue Sep 23, 2018 · 5 comments

Comments

2 participants
@jp112sdl
Contributor

jp112sdl commented Sep 23, 2018

Users can add system variables to the right side of the start page.

These system variables are sorted chronologically how they were added.
Maybe it would be clearer to order them by name.

The easiest way could be by adding the SortByName() method to the enum UserSharedObjects(), that does not work at the moment.

See startpage.htm#L225

Thx to @Schello805 for this suggestion!

@jens-maus

This comment has been minimized.

Owner

jens-maus commented Sep 24, 2018

@jp112sdl Thanks for the suggestion. However, the object UserSharedObjects() returns is formally no Enum, it returns an idarray and for this ReGaHss object type no SortByName() implementation exists yet. While I could of course add this sort function, this would require a new ReGaHss version to be published first. Thus, this could in fact take some time to be actually implemented.

@jens-maus jens-maus changed the title from add SortByName() method to enum UserSharedObjects() to ReGaHss: add SortByName() method to enum UserSharedObjects() Sep 24, 2018

@jens-maus jens-maus added this to the future release milestone Sep 24, 2018

@jp112sdl

This comment has been minimized.

Contributor

jp112sdl commented Sep 24, 2018

I think that is not a problem. The list is unsorted all the years... No need to hurry :)

@jens-maus jens-maus added this to In progress in WebUI improvements/fixes Sep 26, 2018

@jens-maus jens-maus moved this from Done to In progress in ReGaHss improvements/fixes Sep 27, 2018

@jens-maus

This comment has been minimized.

Owner

jens-maus commented Oct 16, 2018

see 8003adb

@jp112sdl

This comment has been minimized.

Contributor

jp112sdl commented Oct 22, 2018

@jens-maus
Hi, kannst du sagen, ob diese Änderung in der ReGaHss.community des OCCU Commit #2c6aeff enthalten ist?
Im Kommentar wird nur auf #441 Bezug genommen.

@jens-maus

This comment has been minimized.

Owner

jens-maus commented Oct 22, 2018

@jp112sdl Ja, das SortByName() ist auch in der Version von OCCU commit
eq-3/occu@2c6aeff mit dabei. In der Tat habe ich das SortByName() in der ReGaHss Version R1.00.0388.0130 (Oct 14 2018) aufgenommen. D.h. ab dann sollte es mit dabei sein.

Allerdings würde ich nicht dazu raten die ReGaHss Version aus diesem Commit zu nehmen. Die ist für die CCU Firmware 3.41.x und später vorbehalten. Dort drin gibt es weitere Änderungen die diese Version (dabei handelt es sich um die ReGaHss R1.00.0388.0200) inkompatibel zu früheren macht und folglich zu Problemen führen kann wenn man dann mit seiner durch die R1.00.0388.0200 gespeicherte regadom datei auf eine frühere Version zurückgehen will. Also am besten bis zum release der 3.41.x version die ReGaHss R1.00.0388.0130 (Oct 14 2018) einsetzen die es im b_37_7 branch von OCCU gibt (https://github.com/jens-maus/occu/tree/b_3_37).

Die neue ReGaHss wird auf jedenfalls teil des kommenden RaspberryMatic updates sein und ich schliesse daher dieses Ticket.

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