Rewrite frontend for the app with React#10
Open
chicco-carone wants to merge 5 commits intoITSpecialist111:mainfrom
Open
Rewrite frontend for the app with React#10chicco-carone wants to merge 5 commits intoITSpecialist111:mainfrom
chicco-carone wants to merge 5 commits intoITSpecialist111:mainfrom
Conversation
Owner
|
Wow - thank you so much for investing time into this. I'm just finishing up work, then will get to testing this evening! |
Author
|
There is currently only one major issue that is the same that was causing some issues when running not in supervisor. that is the hardcoded api url which currently is not working. if you have something in mind to fix it i could implement it easily especially since i'm not sure how networking works between ingress in ha. If you want to quickly try the frontend you can run the dev server on your local machine and edit the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces a significant update to the
automation_inspectorproject by integrating a new frontend built with Next.js and Tailwind CSS, alongside improvements to the backend and Docker setup. The changes enhance the project's architecture, functionality, and developer experience.Frontend Integration and Features:
automation-inspector-frontendfolder, including key files such aslayout.tsx,page.tsx,App.tsx, andAutomationRow.tsx. This frontend provides a modern UI for inspecting Home Assistant automations.globals.cssfor styling consistency and responsiveness.eslint.config.jsandeslint.config.mjs.Backend Updates:
main.pyto dynamically mount frontend directories based on the new Next.js structure, ensuring compatibility with the updated frontend.main.py.Docker and Build Process:
Dockerfileto include a multi-stage build process for the frontend using Node.js, followed by the backend setup. This ensures optimized Docker images..dockerignoreto exclude unnecessary files from the Docker context, such asnode_modulesand logs.These changes collectively modernize the project, streamline the development workflow, and enhance the user experience with a robust and visually appealing interface.
I know that this pr is really big and probably unexpected but after seeing your tool and understanding its usefulness my only issue was that awful looking frontend. If you have any questions about it just tell me and I'll try to explain it.