Skip to content

Use reflex - #11

Merged
leon1995 merged 2 commits into
mainfrom
use-reflex
Aug 25, 2025
Merged

Use reflex#11
leon1995 merged 2 commits into
mainfrom
use-reflex

Conversation

@leon1995

Copy link
Copy Markdown
Owner

No description provided.

@leon1995
leon1995 requested a review from Copilot August 25, 2025 06:48
@leon1995 leon1995 self-assigned this Aug 25, 2025

Copilot AI left a comment

Copy link
Copy Markdown

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 refactors the application from a Qt-based desktop application to a web application using Reflex. The migration includes replacing the PySide6 UI framework with a web-based interface while maintaining the core working time verification functionality. Additionally, it updates the package structure to use the newer factorialhr API client.

Key changes:

  • Migrated from Qt/PySide6 desktop application to Reflex web framework
  • Replaced synchronous API calls with async patterns using httpx and anyio
  • Updated authentication from API key-only to OAuth2 with refresh token support

Reviewed Changes

Copilot reviewed 28 out of 35 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
tests/test_working_time_verification.py New comprehensive test suite for working time verification module
tests/test_verifier.py Removed old verifier test file
src/fwtv/ Removed entire Qt-based desktop application source directory
rxconfig.py Added Reflex configuration file
pyproject.toml Updated dependencies and project structure for web application
factorialhr_analysis/ Added new web application source with pages, components, and verification logic
README.md Updated usage instructions for web application deployment

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

Comment thread factorialhr_analysis/pages/index.py
Comment thread factorialhr_analysis/pages/index.py
@leon1995
leon1995 merged commit f9ec832 into main Aug 25, 2025
2 of 20 checks passed
@leon1995
leon1995 deleted the use-reflex branch August 25, 2025 06:52
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