Skip to content

Prevent invalid quick calculations and clarify calculation history - #172

Open
mdrlzy wants to merge 4 commits into
mainfrom
fix/quick-empty-target-crash
Open

Prevent invalid quick calculations and clarify calculation history#172
mdrlzy wants to merge 4 commits into
mainfrom
fix/quick-empty-target-crash

Conversation

@mdrlzy

@mdrlzy mdrlzy commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

📌 Description

  • Prevent invalid quick calculations from being saved
  • Add ValidateQuickCalculationUseCase as the shared source of validation
  • Use the validation use case for both the Save button state and the final save attempt
  • Log add_quick_invalid_calculation_attempt when an invalid save is attempted
  • Rename the unpinned calculations section to Calculation history

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.

1 participant