-
-
Notifications
You must be signed in to change notification settings - Fork 242
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
Upgrade to latest Vue and Framework7 #2151
Comments
Hello @ThaDaVos, thanks for picking this up. So: Any help with upgrading is highly appreciated. @gyhs WDYT? |
It will mostly be a single-day-a-week effort I can put in, because of work, but at least it will be worked on! I will focus on the |
Sounds great! MainUI is inside the I would recommend to focus on getting MainUI starting with Vue 3 and latest Framework7 first, then check that all features relying on dependencies still work/upgrade the dependencies. I think working on the backend the same time could be quite overwhelming, and if your UI PR does not only contain the upgrade, but also changes to backend calls etc., it will get bigger and bigger and therefore more difficult to review. |
I will start with the pure upgrade first - saw there are tests or at least a framework for it - so will check that out first A question though, why was there chosen for SSE and not Websockets? Just curious |
I don‘t know, that was before my time as maintainer (and even as contributor). |
Ah good to know - when I have time I'll go through the code (created a fork) and sum up what to do in a pull request so we can keep track and verify it's the correct steps |
As the MainUI code needs to be changed anyway when switching from SSE to WebSocket, it could probably be done in a way where the MainUI code itself does not know about the implementation details of the event connection, i.e. whether it is SSE or WebSocket (or probably something different in the future?), this only matter for the implementation of that event connection. In the Java world I would say create an Interface for the event stream and then have two implementations, one for WebSockets and one for SSE. |
That was I thinking too - as I probably have to rewrite stuff to be more Vue 3 - I could also take a small detour and check that out - but first I'll check what components are used and how stuff is done - then I can write up a detailed list of steps and tasks to undertake |
Quick question, maybe I missed it - but is there a contribution guide? |
Yes, there is:
|
|
If you find something missing in the web socket part on server-side, feel free to tag me. |
I just saw these, I have upgraded webpack in these PR #2267 |
I have about 2-3 weeks experience in Javascript / Vue / Framework7, but I'd be happy to help. I have a lot of free time. |
I am first working on indexing all the needed changes and steps to be made - when I've done that, I can may be split off some work - finally getting some time again, so will be picking it up again soon, first of merging the above mentioned PR and then continue indexing the changes - will create a PR with a check list |
@ThaDaVos FYI I upgraded to the latest Vue 2.7.x version and upgraded most dependencies to their latest version compatible with Vue 2.7.x. |
@florian-h05 thank you! Due to work I haven't been able to work on it, so this helps a lot! |
Yeah, this removes some of the burden of upgrading to Vue 3 because most dependencies are now very close to their Vue 3 compatible versions, but I fear it still is a lot of work. |
Yeah, the first step I would have done the same, first upgrade to last minor versions and then go major by major - hope I'll get time soon as I already have experience with Vue 2 to Vue 3 upgrades |
@ThaDaVos Any updates on your work? |
@florian-h05 - slow progress - as I had some really busy days - I hope to be able to give an update within the next two weeks as my schedule is finally clearing up and I finally get some time to tinker with it |
Thanks for the update, sounds good 👍 |
@ThaDaVos any updates? |
@ThaDaVos Still on it? |
Sadly only half - work has been crushing me - been looking into the code and slowly collecting what needs to be changed and how - but sadly no real progress currently - hopefully I can start on it soon and finish it up |
Thanks for the update - do you want to share what needs to be changed in this issue? |
I'll try to give you an update coming weekend if possible due to time |
The problem
It's actually a re-open of an existing issue as it was closed due being stale: #1069
Your suggestion
Check if we can do the upgrade - I am happy to look into this to get Openhab's MainUI up to date.
Your environment
Additional information
The text was updated successfully, but these errors were encountered: