Skip to content

feat: Add edit, confirm, and cancel confirmation to tmpo manual#85

Merged
DylanDevelops merged 1 commit intomainfrom
ravel/manual-confirmation
Feb 18, 2026
Merged

feat: Add edit, confirm, and cancel confirmation to tmpo manual#85
DylanDevelops merged 1 commit intomainfrom
ravel/manual-confirmation

Conversation

@DylanDevelops
Copy link
Owner

Pull Request Checklist

  • I have read and followed the contribution guidelines.
  • My pull request targets the main branch of tmpo.
  • I have tested these changes locally on my machine.

Closes #84

Description

This pull request refactors the manual time entry creation flow in cmd/entries/manual.go to improve user experience and make the process more interactive. The main change is the introduction of a loop that allows users to review a summary of their entry and edit fields before confirming, along with improved handling of prompts and input defaults. The code is also reorganized for clarity and robustness.

User Experience Improvements:

  • Added an interactive loop for manual entry creation, allowing users to review a summary of their entry and choose to confirm, edit, or cancel before finalizing. Entry summary includes project, times, duration, description, milestone, hourly rate, and earnings.
  • Enhanced prompts for all fields (project, start/end date and time, description, milestone) to show previous values as hints, support editing, and preserve user input across edits.

Code Robustness and Organization:

  • Moved database initialization and config loading to the start of the command, ensuring resources are available throughout the entry process.
  • Improved error handling and input validation for prompts, parsing, and milestone selection, providing clearer feedback and preventing invalid entries. [1] [2]

Consistency and Output Formatting:

  • Standardized currency formatting for hourly rate and earnings to always use the global config’s currency setting.

Screenshots

Screenshot 2026-02-18 at 11 19 26 AM

@DylanDevelops DylanDevelops merged commit 8ce15ee into main Feb 18, 2026
6 checks passed
@DylanDevelops DylanDevelops deleted the ravel/manual-confirmation branch February 18, 2026 19:23
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.

Confirmation with all data on tmpo manual before creation allowing for edits

1 participant