-
Notifications
You must be signed in to change notification settings - Fork 6
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
Simple web UI, Automatic builds and stuff #8
base: main
Are you sure you want to change the base?
Conversation
…y server to simulate data
… and have vue-cli installed. audit update
… access later. delay after finding ws server. more info on LCD
…ons for easier use. More logging for loading
@MakerDockio forgot about this request :P :) |
Hey, Vid shows enough thanks :) Okay, so there are only 2 reasons this would happen.
Could you hop onto F12/Developer Tools and see if there are any logs showing up? Thanks |
this is the console output, does show a few errors. i checked the older version of UI, only shows the websocket data, but no direct errors on the log.
|
…to relay release download link
After much confusion, I have seen what is cooking. You weren't using the release from this merge, but rather the one from master. Please test the release from here: https://github.com/mrinc/UT61EWIFI/releases Let me know if there is still an issue? |
Deleted release file.
@jonoxer Please give the release a test and see if it works for you as well? The added includes a build so it will compile the NodeRED code and desktop apps. |
@mrinc I've made some considerable UI changes based on the main branch but haven't yet pushed them (been out of action for a while). Seems your latest PR has some considerable structure changes to the project. I will pull down and review and see how plausible it is to make the UI changes I have locally today and give it all a test. All going well ill push the changes to the main branch for everyone to benefit. |
@MakerDockio Glad to here you're still with us :) SuperHouse main branch doesn't have the project (simple web) at all. The project as compiled should be basically the same.
And ( So doing development on the UI will be much easier. The site itself has a lot more fixes than what is in master as well, so give it a spin and go from there :)
In the web project:
|
@MakerDockio Your changes have been merged :) |
@MakerDockio Multi-platform builds are fully setup. I have done as much testing as I can on my end and all does seem to be functioning as expected.
I think we should look at closing this merge off so smaller commits can be made as required going forward :) |
@MakerDockio @austinscreations @jonoxer ... I think it is time to merge this haha :P |
i don't have merge capabilities. and haven't tested any of the new stuff yet myself... |
Will try and allocate some time this week to give this a test with the device. All being well i'll get it merged in. |
@MakerDockio Just a reminder, I know things have been chaos :) |
@austinscreations
|
UI based from @MakerDockio now in a full vue project for easy maint.
node-red is automatically compiled from the simpleWeb project.
Releases are automatically created with all the clients.
SimpleWeb:
Contains themes, graphs and stuff...
Desktop client:
Simple electron app
CI/CD:
Builds are automatically versioned and packaged ready for use with github ci