Skip to content

fix(oceancurrent): rename sealCtd productId and align tests#1262

Merged
shaunahu merged 1 commit intomasterfrom
bugfix/7296-seal-ctd-tracks-product-id-mismatch
Aug 15, 2025
Merged

fix(oceancurrent): rename sealCtd productId and align tests#1262
shaunahu merged 1 commit intomasterfrom
bugfix/7296-seal-ctd-tracks-product-id-mismatch

Conversation

@weited
Copy link
Contributor

@weited weited commented Aug 15, 2025

  • Add .venv/ to .gitignore

@weited weited requested review from Copilot and shaunahu August 15, 2025 06:44
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR renames the sealCtd product identifiers from "sealTrack" to "sealTracks" (adding an 's') to align naming conventions across the oceancurrent file server API and its corresponding tests.

  • Updates product IDs from singular "sealTrack" to plural "sealTracks" form
  • Aligns test cases to match the renamed product identifiers
  • Ensures consistency between API configuration and test verification

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
oceancurrent_file_server_api.py Updates product IDs from "sealCtd-sealTrack" to "sealCtd-sealTracks" and "sealCtd-sealTrack-video" to "sealCtd-sealTracks-video"
test_oceancurrent_file_server_api.py Updates test cases and verification calls to use the renamed product IDs with plural "sealTracks"

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Copy link
Contributor

@shaunahu shaunahu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shaunahu shaunahu merged commit e66ba9a into master Aug 15, 2025
0 of 2 checks passed
@shaunahu shaunahu deleted the bugfix/7296-seal-ctd-tracks-product-id-mismatch branch August 15, 2025 06:56
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.

3 participants