Skip to content

enhance startup UX showing opening POS widget#383

Merged
bishoy-at-pieces merged 1 commit intomainfrom
enhance-ux-startup
Jul 29, 2025
Merged

enhance startup UX showing opening POS widget#383
bishoy-at-pieces merged 1 commit intomainfrom
enhance-ux-startup

Conversation

@bishoy-at-pieces
Copy link
Collaborator

Screen.Recording.2025-07-25.at.7.35.33.PM.mov
Screenshot 2025-07-25 at 7 35 11 PM

Copilot AI review requested due to automatic review settings July 25, 2025 16:37
Copy link
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 enhances the startup user experience by adding a visual progress indicator when opening the PiecesOS widget. It shows a spinner with descriptive text during the opening process and provides clearer feedback when PiecesOS is not installed.

  • Adds a progress indicator with spinner and status messages during PiecesOS widget opening
  • Improves error handling in the subprocess calls for opening PiecesOS across different platforms
  • Refactors startup flow to use the new progress-enabled widget opening method

Reviewed Changes

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

File Description
src/pieces/settings.py Adds open_pieces_widget() method with progress indicator and updates startup logic to use it
src/pieces/_vendor/pieces_os_client/wrapper/client.py Improves error handling by checking subprocess return codes and adding platform validation

@bishoy-at-pieces bishoy-at-pieces merged commit 9cb81b3 into main Jul 29, 2025
7 checks passed
@bishoy-at-pieces bishoy-at-pieces deleted the enhance-ux-startup branch July 29, 2025 21:29
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.

[PERFORMANCE] CLI takes to much to show the Install PiecesOS if it is not installed

3 participants