lolwheel / Owie

MIT License
145 stars 43 forks source link

Complete rewrite of the Webinterface #76

Open mretich opened 10 months ago

mretich commented 10 months ago

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

Kristowsa commented 10 months ago

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

mretich commented 6 months ago

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 commented 5 months ago

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

mretich commented 5 months ago

@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 commented 5 months ago

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 commented 5 months ago

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

Thanks!

mretich commented 5 months ago

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 commented 5 months ago

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 commented 4 months ago

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.