-
Notifications
You must be signed in to change notification settings - Fork 2
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
dashboard design #2
Comments
Hey @Geal, i'd love to contribute to the frontend. But i'd rather put it into it's own repository, make a webpack/babeljs pipeline and use reactjs. |
why in another repository? I'm not really attached to the current frontend code (I mean, look at it :D), so it should be fine.
|
Another repository because it would come with it's own unit tests and build steps. The produced artifact would be a single js file with all needed ressources embedded into it. just About the data management, it absolutely should be async and at best should use the same API accessible to everyone using it. My usual aproach is axios and redux, if data changes come in too frequent for redux then build something new with the react context api. |
started building it at https://github.com/Vengarioth/grad-frontend can transfer the repo to you if you like |
it can live there for now, it's fine. How would you integrate that with the cargo build? Considering that the file should be integrated when building, even in a |
i would commit the compiled js file into this repo, it should only be updated with new releases of the frontend project, at some point we can publish the frontend on either npm or cargo(packaged in a rust lib). Edit: but the heavy lifting / build process into a rust accessible project should be in the other repo. |
alright then :) |
there is none :D
Just an accumulation of graphs. Let's find a way to make it look nice! I'm open to any idea and pull requests here. There's only one thing that will stay for now: using chartjs to display the data (for performance reasons). But any graphical changes to the graphs are fair game :)
The text was updated successfully, but these errors were encountered: