Skip to content

Sync OpenSplitTime split entry kinds before upload - #4

Merged
rleikis merged 6 commits into
mainfrom
copilot/refer-to-this-pull-request
Sep 18, 2026
Merged

Sync OpenSplitTime split entry kinds before upload#4
rleikis merged 6 commits into
mainfrom
copilot/refer-to-this-pull-request

Conversation

Copilot AI commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Synchronize OpenSplitTime event-group split entry kind metadata in the uploader and use it to filter generated raw times so each station only sends allowed in/out records.

Changes

  • fetch the event first and update the uploader's event-group identifier from the event relationship before hydrating split-kind metadata
  • extend the OpenSplitTime response models for event-group entry metadata
  • filter raw-time JSON generation by the mapped split's allowed sub-split kinds, with fallback to both kinds only when a split is absent from OST metadata
  • surface a ready-state warning when split-kind synchronization is unavailable or the event group cannot be resolved
  • guard against unsupported station headers

Validation

  • dotnet build ost-uploader.sln -p:EnableWindowsTargeting=true
  • secret scanning on changed files
  • CodeQL previously passed with 0 alerts
  • final post-fix parallel validation could not complete due time limit

Copilot AI and others added 4 commits September 15, 2026 10:23
@rleikis
Add OST split kind filtering
ca58ae8 
Co-authored-by: rleikis <10184069+rleikis@users.noreply.github.com>
@rleikis
Fix split kind fallback behavior
6f9f41e 
Co-authored-by: rleikis <10184069+rleikis@users.noreply.github.com>
@rleikis
Use event event-group metadata
cf8d606 
Co-authored-by: rleikis <10184069+rleikis@users.noreply.github.com>
@rleikis
Guard missing OST event group
77bf2a0 
Co-authored-by: rleikis <10184069+rleikis@users.noreply.github.com>
Copilot AI requested a review from rleikis September 15, 2026 10:36
@rleikis rleikis linked an issue Sep 16, 2026 that may be closed by this pull request
Pushing in and out times causes #3
Closed
@rleikis rleikis added the bug Something isn't working label Sep 16, 2026
@rleikis

rleikis commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Copilot AI and others added 2 commits September 18, 2026 19:21
@rleikis
Merge remote-tracking branch 'origin/main' into copilot/refer-to-this…
5a94775 
…-pull-request

# Conflicts:
#	ost-uploader/MainWindow.xaml.cs

Co-authored-by: rleikis <10184069+rleikis@users.noreply.github.com>
@rleikis
Harden CI workflow permissions
a03d4b0 
Co-authored-by: rleikis <10184069+rleikis@users.noreply.github.com>

Copilot AI commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request

Resolved in 5a94775.

@rleikis
rleikis marked this pull request as ready for review September 18, 2026 19:27
@rleikis
rleikis merged commit 2bf5024 into main Sep 18, 2026
1 check passed
@rleikis
rleikis deleted the copilot/refer-to-this-pull-request branch September 18, 2026 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rleikis rleikis Awaiting requested review from rleikis

Assignees

@rleikis rleikis

Copilot code review Copilot

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Pushing in and out times causes

2 participants

@rleikis