Skip to content

Add ability to save/view exercise history#5

Merged
brylie merged 9 commits intomainfrom
exercise-history-dexie
Mar 8, 2025
Merged

Add ability to save/view exercise history#5
brylie merged 9 commits intomainfrom
exercise-history-dexie

Conversation

@brylie
Copy link
Copy Markdown
Owner

@brylie brylie commented Mar 8, 2025

This pull request includes several changes to improve the project structure, add new features, and enhance testing. The most important changes include updating the package management and directory structure, adding end-to-end tests, integrating Dexie for database management, and refining the TypeScript types.

Project Structure and Configuration:

  • Updated .github/copilot-instructions.md to reflect the use of pnpm for package management and specified that commands should be run from the app/ directory.
  • Added a new README.md in the .github/copilot/ directory to document the GitHub Copilot configuration settings.
  • Created options.json to store Copilot-specific settings, including the workspace path, package manager, import style, and testing command.
  • Updated .vscode/settings.json to set the terminal working directory to app/, exclude unnecessary files from search and file watching, and configure TypeScript SDK path.

End-to-End Testing:

  • Added comprehensive end-to-end tests for the workout generator and history flow using Playwright in app/e2e/workout-history.test.ts.

Database Integration:

  • Integrated Dexie for IndexedDB management, including defining the schema and adding functions for saving, retrieving, and deleting completed exercises in app/src/lib/database.ts.
  • Added tests for the database functions to ensure proper saving and retrieval of completed exercises in app/src/lib/database.test.ts.

TypeScript Types:

  • Updated TypeScript types to include a new CompletedExercise interface and added a completed property to the WorkoutItem interface in app/src/lib/types.ts. [1] [2]
  • Adjusted tests to reflect the updated types in app/src/lib/types.test.ts. [1] [2]

Dependency Management:

  • Added dexie and fake-indexeddb dependencies to app/package.json and updated the pnpm-lock.yaml file accordingly. [1] [2] [3] [4] [5] [6] [7] [8]

@brylie brylie changed the title Exercise history dexie Add ability to save/view exercise history Mar 8, 2025
@brylie brylie merged commit 2e05fe4 into main Mar 8, 2025
@brylie brylie deleted the exercise-history-dexie branch March 24, 2025 16:45
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