Skip to content

feat: add View History controls for editing and deleting sessions#27

Open
despossivel wants to merge 1 commit into
sooryathejas:mainfrom
despossivel:main
Open

feat: add View History controls for editing and deleting sessions#27
despossivel wants to merge 1 commit into
sooryathejas:mainfrom
despossivel:main

Conversation

@despossivel

Copy link
Copy Markdown

Description
This PR improves the CLI View History flow by adding direct control over history records.

Users can now:

  1. View session details by SL#
  2. Edit history entry fields (target and status)
  3. Delete a full session by SL# directly from the history menu

What Changed

  1. Refactored View History into an action-oriented submenu.
  2. Added history-level DB operations:
  3. Fetch history row by SL#
  4. Update history entry by allowed field (target/status)
  5. Reused full-session deletion in the new history flow
  6. Added input validation for:
  7. Invalid SL#
  8. Invalid edit field
  9. Empty update values

Impact

  1. Better history usability without jumping across multiple flows.
  2. Faster correction of target/status entries.
  3. Centralized sensitive maintenance actions inside View History.

Compatibility

  1. No database schema breaking changes.
  2. Existing session-level edit/delete flows remain intact.

How to Validate

  1. Open View History.
  2. Test viewing a session by SL#.
  3. Test editing target and status for an existing session.
  4. Test invalid SL# and invalid field handling.
  5. Test full session deletion and confirm removal from history list.

Technical Validation

  1. Python syntax compilation passed for core modules.
  2. No static analysis errors reported in updated files.

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