feat: add vessel_tags filter to Vessels.search()#570
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds vessel tag filtering support to the Vessels.search() reference endpoint, aligning it with tag-filter patterns used across other endpoints and releasing the change as version 1.0.29.
Changes:
- Added
vessel_tagsandvessel_tags_excludedparameters toVessels.search(). - Passed these parameters through in the request payload.
- Added unit tests covering single/multiple tag inclusion and exclusion; bumped SDK version to
1.0.29.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
vortexasdk/version.py |
Bumps SDK version to 1.0.29. |
vortexasdk/endpoints/vessels.py |
Adds vessel_tags/vessel_tags_excluded parameters and includes them in api_params. |
tests/endpoints/test_vessels.py |
Adds unit tests asserting the new params are passed through to the client call. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
e7ed8c5 to
f8c1b14
Compare
…rch() Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
f8c1b14 to
4f5115b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
RELATED TICKETS
CHANGELOG
vessel_tagsandvessel_tags_excludedparameters toVessels.search()to support filtering vessels by tags (e.g.vessel_fso_tag,vessel_decommissioned_tag)TESTS
COMMENTS
USAGE EXAMPLE
Install from this branch:
Then use:
Or run the included script from a local clone of this branch:
🤖 Generated with Claude Code