mobtimeapp / mobtime-web

Web front-end of mobtime
https://mobti.me/
0 stars 2 forks source link

Import existing MobTime UI #5

Closed JStruk closed 1 year ago

JStruk commented 2 years ago

Import existing UI code from https://github.com/mobtimeapp/mobtime/tree/master/public

Description

Review Notes

This PR is large since most of it is copy/paste from the original mobtime repo

Some important-ish files to review would be:

Smoke Test

The app can be run as a local dev server without rollup doing its thing for easy debugging, and can also be built for production with the output from Vite's rollup going to dist/

Spinning up a local dev server

npm run dev should start a vite server @ localhost:5173

Building for production

npm run build produces a bundle via Vite for production. Outputs bundle to dist/

Notes

Websocket-related code fails right now. Example: press the 'START TURN' button and look at the console. How do we proceed with this?

ToDo

Before this is out of draft:

mrozbarry commented 1 year ago

@JStruk npm run tailwind fails for me, but it's easy to fix. Can you change the file extension to .cjs and update package.js script "tailwindcss build -i ./src/styles/index.css -o ./src/styles/styles.css -c ./tailwind.config.cjs" (note the extension update).

JStruk commented 1 year ago

@JStruk npm run tailwind fails for me, but it's easy to fix. Can you change the file extension to .cjs and update package.js script "tailwindcss build -i ./src/styles/index.css -o ./src/styles/styles.css -c ./tailwind.config.cjs" (note the extension update).

Nice catch. Updated and tested locally as well 👍