Skip to content
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

Complete rewrite of the Webinterface #76

Open
wants to merge 30 commits into
base: main
Choose a base branch
from
Open

Conversation

mretich
Copy link
Contributor

@mretich mretich commented Jan 10, 2024

I have completely rewritten the Web Interface, to a more modern approach and style.
It's completely vanilla js based with XHR (JSON) backend communication.

Since the currently used minifier was not able to handle the JS, CSS, and HTML files correctly,
I needed to introduce node.js and the "minify" package.
So for the build to work, you need to have node and npm (or the package manager you like) installed and the minify package defined in package.json
(Will be installed automatically if "npm install" is triggered in the source folder)
All features are implemented, and are tested.
(Onewheel XR)

And Yes, I am aware that this project is pretty much abandoned. :(

P.S.: some images of the new Interface are attached :)
Owie_new_03
Owie_new_02
Owie_new_01

Michael Retich and others added 25 commits October 6, 2022 00:02
… to match the new frontend

apply new index.html, styles.css and add new owie.js, move to 'minify' in gen_data.py to handle the js file correctly, adjust platform io upload speed
…bsulete files, add minify to package.json so npm install will install this dep.
fix default value handling, refactor /update fallback sites, cade cleanup
…essbar icon not chenged in overlay div

code cleanup, changed order of displayed stats
…lean up, restore accidentally removed css property
- refactor deprecated code
- adjust stylesheet for new icons
- fixed error while minifying under windows
- adjusted temperatur unit labels
- add visual severity to temperature gauges
 - adjust battery icon size and position
…settings page, fix error shown while firmware update, pushed to OWIE version 2.1.0
@Kristowsa
Copy link

Nice one I'll have a look when I get the xr back up and running

@mretich
Copy link
Contributor Author

mretich commented Apr 21, 2024

Nice one I'll have a look when I get the xr back up and running
@Kristowsa
May I ask if you have already take a look?
BTW: If you have any suggestions on the Interface, don´t bother to open a issue at my forked repo ;)

@Jared-Is-Coding
Copy link
Contributor

@mretich Have you had anyone testing this? I'd be up for it potentially.

@mretich
Copy link
Contributor Author

mretich commented Jun 11, 2024

@mretich Have you had anyone testing this? I'd be up for it potentially.

Actually I have this running on my own XR, and a college has it installed on his 2 Pints.
So, yes it is tested :)

BUT: I used my own repository version from https://github.com/mretich/Owie/releases/tag/2.1.0
(Pretty much the same, since main was merged before pull request)

@Jared-Is-Coding
Copy link
Contributor

BUT: I used my own repository version from https://github.com/mretich/Owie/releases/tag/2.1.0 (Pretty much the same, since main was merged before pull request)

No biggie. It was just some updates to the README. Would be pretty easy to merge in if you wanted.

I'll drop this on my board as well!

@Jared-Is-Coding
Copy link
Contributor

Is there instruction for compiling the codebase? I only see mention of the installation of the minify package.

Thanks!

@mretich
Copy link
Contributor Author

mretich commented Jun 11, 2024

Is there instruction for compiling the codebase? I only see mention of the installation of the minify package.

Thanks!

You will need NodeJS >= 16 and a NodeJS package manager (e.g. NPM) + all other requirements used for the original OWIE compiling process.
Before the first compiling run, you will need to run 'npm install' in the root folder of the checked out repo, which will install the required packages. (hopefully).
(you should see a 'node_modules' folder after install)
After that the process is the same as the original OWIE compile process.

Regards

@Jared-Is-Coding
Copy link
Contributor

After that the process is the same as the original OWIE compile process.

Right, this is the part I'm unfamiliar with. Is it outlined somewhere in the repository that I've missed? Thanks again!

Jared-Is-Coding and others added 5 commits June 20, 2024 10:44
added a facebook discussion group, since I found now place where the community would discuss owie issues
- Add image files for instruction
- Update instructions to reference images
Update installation instruction imagery, Add Facebook link, Add tips and warnings
@Jared-Is-Coding
Copy link
Contributor

Jared-Is-Coding commented Jun 24, 2024

FYI There's a few lines in the README that I updated to point to your fork that probably don't belong in a PR to the lolwheel main @mretich

In other news, I've just installed this and will start testing with it on my VESC.

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.

4 participants