You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Consolidated resource mixins: Moved ActivatableMixin, PublishableMixin, DocumentMixin, and MediaMixin (and their async variants) from domain-specific locations (catalog, program, integration, accounts) to a centralized mpt_api_client.resources.mixins module
Introduced ReviewableMixin: Added new ReviewableMixin and AsyncReviewableMixin classes providing a review() method for resources
Added review capabilities: Extended the following service classes with ReviewableMixin/AsyncReviewableMixin:
ItemsService and AsyncItemsService (catalog)
TermVariantService and AsyncTermVariantService (catalog)
TermService and AsyncTermService (catalog)
ProductsService and AsyncProductsService (catalog)
DocumentService and AsyncDocumentService (catalog products documents)
MediaService and AsyncMediaService (catalog products media)
Updated imports: Updated all modules across catalog, program, integration, and accounts resources to import mixins from the new centralized location
Removed duplicate definitions: Eliminated duplicate mixin definitions from catalog, program, and integration specific mixin modules
Simplified docstrings: Shortened docstrings for publish() and unpublish() methods
Updated unit tests: Refactored and consolidated test files to use centralized mixin imports and added comprehensive tests for the new ReviewableMixin functionality
Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.
Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.
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
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.
Closes MPT-20504
Release Notes
Consolidated resource mixins: Moved
ActivatableMixin,PublishableMixin,DocumentMixin, andMediaMixin(and their async variants) from domain-specific locations (catalog,program,integration,accounts) to a centralizedmpt_api_client.resources.mixinsmoduleIntroduced ReviewableMixin: Added new
ReviewableMixinandAsyncReviewableMixinclasses providing areview()method for resourcesAdded review capabilities: Extended the following service classes with
ReviewableMixin/AsyncReviewableMixin:ItemsServiceandAsyncItemsService(catalog)TermVariantServiceandAsyncTermVariantService(catalog)TermServiceandAsyncTermService(catalog)ProductsServiceandAsyncProductsService(catalog)DocumentServiceandAsyncDocumentService(catalog products documents)MediaServiceandAsyncMediaService(catalog products media)Updated imports: Updated all modules across
catalog,program,integration, andaccountsresources to import mixins from the new centralized locationRemoved duplicate definitions: Eliminated duplicate mixin definitions from catalog, program, and integration specific mixin modules
Simplified docstrings: Shortened docstrings for
publish()andunpublish()methodsUpdated unit tests: Refactored and consolidated test files to use centralized mixin imports and added comprehensive tests for the new
ReviewableMixinfunctionality