Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add is removed status (TS-2275) #1084

Merged
merged 4 commits into from
Jan 21, 2025
Merged

Add is removed status (TS-2275) #1084

merged 4 commits into from
Jan 21, 2025

Conversation

Roffenlund
Copy link
Contributor

  • Added an is_removed property to the Package and PackageVersion models to determine whether the respective packages and package versions are marked as removed.

  • Included the is_removed field in the DependencySerializer.

  • Implemented corresponding unit tests.

@Roffenlund Roffenlund self-assigned this Jan 10, 2025
Implement a is_removed property in the Package model that determines
whether a package is removed by checking if is_active is set to False
or if the Package has no active versions.

Implement a is_removed property in PackageVersion that determines
whether a package version is removed by 1) checking if the related
Package is removed 2) checks if is_active is set to False.

Refs. TS-2275
Add a boolean field to DependecySerializer with the source
set to PackageVersion's is_removed property.

Refs. TS-2275
Implement tests for is_removed properties in Package and
PackageVersion.

Refs. TS-2275
@Roffenlund Roffenlund force-pushed the add-is-removed-status branch from 5d3993f to 5a07af2 Compare January 16, 2025 12:49
Remove the source property from the is_removed BooleanField
as it is not required.

Implement an additional test for testing the DependencySerializer's
endpoint.

Refs. TS-2275
@Roffenlund Roffenlund force-pushed the add-is-removed-status branch from 5a07af2 to 32ea076 Compare January 16, 2025 13:11
@MythicManiac MythicManiac merged commit b5cd82f into master Jan 21, 2025
24 checks passed
@MythicManiac MythicManiac deleted the add-is-removed-status branch January 21, 2025 14:10
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.

2 participants