Skip to content

Add progress bar when connecting to drone#722

Merged
1Blademaster merged 3 commits intorelease-alpha-0.1.10from
alpha-0.1.10/721-add-progress-bar-when-connecting-to-drone
Sep 27, 2025
Merged

Add progress bar when connecting to drone#722
1Blademaster merged 3 commits intorelease-alpha-0.1.10from
alpha-0.1.10/721-add-progress-bar-when-connecting-to-drone

Conversation

@1Blademaster
Copy link
Copy Markdown
Member

@1Blademaster 1Blademaster commented Sep 27, 2025

image

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a progress bar to the drone connection process to provide visual feedback to users during the connection sequence. The changes implement a structured approach to tracking connection phases and displaying progress percentage.

  • Updated connection status tracking to include both message and progress data
  • Modified backend to emit progress percentages based on connection phase completion
  • Enhanced frontend UI to display an animated progress bar during connection

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
radio/app/utils.py Adds TypedDict for connection data structure and updates callback to pass structured data
radio/app/drone.py Implements connection phases list and progress calculation logic
gcs/src/redux/slices/droneConnectionSlice.js Updates connection status state to include message and progress fields
gcs/src/redux/middleware/socketMiddleware.js Modifies socket listener to handle structured connection status data
gcs/src/components/navbar.jsx Adds progress bar component and prevents modal closure during connection

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread radio/app/drone.py
Comment thread gcs/src/components/navbar.jsx Outdated
@1Blademaster 1Blademaster merged commit d97964c into release-alpha-0.1.10 Sep 27, 2025
7 checks passed
@1Blademaster 1Blademaster deleted the alpha-0.1.10/721-add-progress-bar-when-connecting-to-drone branch September 27, 2025 18:02
1Blademaster added a commit that referenced this pull request Sep 28, 2025
* Add progress bar when connecting to drone

* Address copilot review comments

* Update progress bar styling
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.

3 participants