Skip to content

feat: shot review phase 2 - #39

Merged
StuartFerguson merged 1 commit into
mainfrom
codex/shot-review-phase-2
Jul 20, 2026
Merged

feat: shot review phase 2#39
StuartFerguson merged 1 commit into
mainfrom
codex/shot-review-phase-2

Conversation

@StuartFerguson

@StuartFerguson StuartFerguson commented Jul 20, 2026

Copy link
Copy Markdown
Owner

What changed

  • Added a shot review workspace with a master-detail layout.
  • Added searchable, session-scoped shot review with included-only filtering.
  • Added bulk include, bulk exclude, club edit, and swing-type edit actions.
  • Exposed raw import data and selected shot details for correction work.
  • Wired the review workspace into the main shell navigation.

Why

Validation

  • dotnet build CarryIQ.sln -c Debug
  • dotnet test tests\\CarryIQ.UnitTests\\CarryIQ.UnitTests.csproj -v minimal
  • dotnet test tests\\CarryIQ.IntegrationTests\\CarryIQ.IntegrationTests.csproj -v minimal

@StuartFerguson
StuartFerguson marked this pull request as ready for review July 20, 2026 10:47
@StuartFerguson StuartFerguson linked an issue Jul 20, 2026 that may be closed by this pull request
[v0.1] Manual shot entry and shot review #5
Closed
@StuartFerguson
StuartFerguson merged commit 8c22db5 into main Jul 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[v0.1] Manual shot entry and shot review

1 participant

@StuartFerguson