Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

重构:使用PlanStepStatus枚举替代硬编码status #422

Merged
merged 9 commits into from
Mar 10, 2025

Conversation

rxdaozhang
Copy link
Collaborator

This pull request introduces a new PlanStepStatus enum class to standardize the statuses of plan steps and updates the codebase to use this enum instead of hardcoded status strings. The changes improve code readability and maintainability by centralizing the status definitions and related utility methods.

Key changes include:

Introduction of PlanStepStatus Enum:

  • app/flow/base.py: Added PlanStepStatus enum class with possible statuses (NOT_STARTED, IN_PROGRESS, COMPLETED, BLOCKED) and utility methods (get_all_statuses, get_active_statuses, get_status_marks).

Refactoring to Use PlanStepStatus:

  • Feature 1
  • Feature 2

Feature Docs

Influence

Result

Other

@mannaandpoem mannaandpoem merged commit d9418c1 into mannaandpoem:main Mar 10, 2025
1 of 2 checks passed
@mannaandpoem
Copy link
Owner

LGTM

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