flownexus-lwm2m / flownexus

IoT server with LwM2M support
https://flownexus-lwm2m.github.io/flownexus
Apache License 2.0
3 stars 2 forks source link

frontend: Add dashboard templates, version 0.1.0 without server functionality #40

Closed Kappuccino111 closed 3 months ago

Kappuccino111 commented 5 months ago

This commit adds the basic dashboard templates for the various dashboards. It also adds a new endpoint for temperature data and updates the URL configuration accordingly.

jonas-rem commented 5 months ago

AdminLTE looks good and looks like it's actively maintained.

Their License hint is a little concerning, but I don't think that the risk of a re-licensing to a proprietary license is high. I have not seen them using Contributor License Agreements (CLAs) for contributors, so they would have to get the permission from all contributors (which hold the copyright) in case of a future license change. So all good from my side:

AdminLTE is an open source project by AdminLTE.io that is licensed under MIT. AdminLTE.io reserves the right to change the license of future releases.

Kappuccino111 commented 4 months ago

@jonas-rem made all the changes as requested. The base frontend is good to go πŸ‘

Kappuccino111 commented 4 months ago

I am attaching a few screenshots here for easier review:

image image image image
jonas-rem commented 4 months ago

@Kappuccino111 as discussed, I merged the backend PRs that have been open. As those renamed some database resources (e.g. device -> endpoint) you have to rebase this PR and update the references.

Kappuccino111 commented 4 months ago

@jonas-rem Yep, I now have some active conflicts to resolve after merging of other PRs. I will resolve them along with the additional implementation as discussedπŸ˜πŸ‘

Kappuccino111 commented 3 months ago

@jonas-rem the following changes have been made in this PR for the first version of the frontend.

This should be merge-ready now πŸ‘πŸ˜„

Kappuccino111 commented 3 months ago

The server-side functionality (such as buttons for OTA, graph updates, and selection) will be implemented in the next PR. However, that PR can only be reviewed once we address the testing issue with POSIX frameworks, which currently prevents running the simulation.py script.

Kappuccino111 commented 3 months ago

@jonas-rem the graph functionality now works according to the latest schema. However, since the initial frontend PR was limited to a single client configuration, I have restricted the current setup to filter only based on resource_name, which doesn’t account for different clients.

Should be ready to merge.... Lesss goooo πŸ˜πŸŽ‰

Kappuccino111 commented 3 months ago

@jonas-rem The branch should be good for merging and release now. Squashing all commits into one would have been quite challenging, as it would require resolving conflicts from previous merges, so I left the history as it is. I have fixed all the failing tests and pushed the latest changes. The lines of code are now back to 1300. Everything is good to go! πŸ‘πŸ˜„

Kappuccino111 commented 3 months ago

@jonas-rem merging this would successfully finish all tasks for release v0.1.0. Less gooo πŸŽ‰πŸŽ‰

Kappuccino111 commented 3 months ago

Closing. Please catch updates on #68.