Skip to content

fix(LAB-4094): add a function to skip or unskip an asset#1991

Closed
stchabi wants to merge 1 commit intomainfrom
feature/lab-4094-aau-i-can-skip-or-unskip-an-asset-with-the-sdk
Closed

fix(LAB-4094): add a function to skip or unskip an asset#1991
stchabi wants to merge 1 commit intomainfrom
feature/lab-4094-aau-i-can-skip-or-unskip-an-asset-with-the-sdk

Conversation

@stchabi
Copy link
Copy Markdown
Contributor

@stchabi stchabi commented Nov 24, 2025

No description provided.

@stchabi stchabi force-pushed the feature/lab-4094-aau-i-can-skip-or-unskip-an-asset-with-the-sdk branch 2 times, most recently from 25c8468 to 4b8274c Compare November 24, 2025 08:27
@stchabi stchabi self-assigned this Nov 24, 2025
Copy link
Copy Markdown
Contributor

@RuellePaul RuellePaul left a comment

Choose a reason for hiding this comment

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

LGTM, just some small docstrings fixes

Copy link
Copy Markdown
Collaborator

@baptiste-olivier baptiste-olivier left a comment

Choose a reason for hiding this comment

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

I think we want to target release/25.2 branch

@stchabi stchabi force-pushed the feature/lab-4094-aau-i-can-skip-or-unskip-an-asset-with-the-sdk branch from 4b8274c to b90e9db Compare November 24, 2025 09:20
@stchabi stchabi changed the base branch from main to release/25.2 November 24, 2025 09:21
@stchabi stchabi changed the base branch from release/25.2 to main November 24, 2025 09:22
- assign(): Assign assets to labelers
- update_priority(): Update asset priorities
- skip(): Skip an asset
- unskip: Unskip an asset
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

missing parenthesis

)
"""
return self._client.skip_or_unskip(
asset_id=asset_id, reason=reason, action="skip", project_id=project_id
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

sort alphabetically when you can

@stchabi stchabi closed this Nov 24, 2025
@baptiste-olivier baptiste-olivier deleted the feature/lab-4094-aau-i-can-skip-or-unskip-an-asset-with-the-sdk branch December 4, 2025 10:00
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