Skip to content

Conversation

@adityashibu
Copy link
Member

This pull request introduces several significant changes to the backend and frontend components of the application, focusing on group management, device status handling, and UI improvements. The most important changes include the addition of group management functionalities, modification of device status handling, and some UI enhancements.

Backend Enhancements:

  • Group Management:

    • Added new endpoints for managing device groups, including adding, editing, deleting, and updating group statuses in backend/fastAPI.py.
    • Implemented group-related functions in backend/groups.py to handle group operations such as adding, deleting, editing, and changing group statuses.
    • Updated database/users_db.json to include sample device groups for users. [1] [2] [3]
  • Device Status Handling:

    • Modified changeDeviceStatus function in backend/devices_json.py to accept a status parameter, allowing for more flexible status updates.
    • Updated the change_device_status endpoint in backend/fastAPI.py to handle the new status parameter.

Frontend Enhancements:

  • UI Improvements:

    • Enhanced the styling of dialog titles and buttons in frontend/src/app/automations/page.jsx to improve the user interface. [1] [2]
  • Code Formatting:

    • Improved code readability by adjusting formatting in frontend/src/app/dashboard/page.jsx. [1] [2] [3]

Minor Changes:

  • Automation Trigger Time Update:

    • Updated the trigger time for an automation in backend/automations.json.
  • Code Cleanup:

    • Removed unnecessary comments and adjusted default parameter settings in backend/automations.py.
    • Commented out unused test code in backend/tests/test_fastAPI.py to clean up the test file.

adityashibu and others added 25 commits March 15, 2025 22:57
…ted user and update documentation for clarity
@adityashibu adityashibu requested a review from david-g-f March 16, 2025 14:15
Copy link
Contributor

@david-g-f david-g-f left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good

@david-g-f david-g-f merged commit 30d55a3 into PowerHouse-Project:main Mar 16, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants