Skip to content

Conversation

@robcsegal
Copy link
Contributor

Added create and update with icon mixin and updated product.
Will be used for accounts and sellers at least.
https://softwareone.atlassian.net/browse/MPT-15309

@github-actions
Copy link

github-actions bot commented Nov 10, 2025

Warnings
⚠️

This PR changes 623 lines across 9 files (threshold: 600). Please consider splitting it into smaller PRs for easier review.

⚠️

Some newly added source files do not have corresponding tests in the tests/ folder with matching structure and the test_ prefix.

✅ Found Jira issue key in the title: MPT-15309

Tests mirroring check (created files only)

Added source file Expected test (added in this PR)
mpt_api_client/constants.py tests/test_constants.py

Generated by 🚫 dangerJS against 1c37146

@robcsegal robcsegal force-pushed the MPT-15309-implement-mixins-for-creating-and-updating-resource-with-image-file branch 3 times, most recently from c0a6f96 to beb6258 Compare November 10, 2025 15:54
@robcsegal robcsegal force-pushed the MPT-15309-implement-mixins-for-creating-and-updating-resource-with-image-file branch from beb6258 to f68c2cc Compare November 10, 2025 16:21
@robcsegal robcsegal requested a review from albertsola November 10, 2025 16:26
@robcsegal robcsegal force-pushed the MPT-15309-implement-mixins-for-creating-and-updating-resource-with-image-file branch from f68c2cc to d2baab9 Compare November 10, 2025 17:18
@robcsegal robcsegal force-pushed the MPT-15309-implement-mixins-for-creating-and-updating-resource-with-image-file branch 2 times, most recently from 023404a to cc75cc2 Compare November 11, 2025 12:43
@robcsegal robcsegal requested a review from d3rky November 11, 2025 12:43
@robcsegal robcsegal force-pushed the MPT-15309-implement-mixins-for-creating-and-updating-resource-with-image-file branch from cc75cc2 to 1c37146 Compare November 11, 2025 12:44
@sonarqubecloud
Copy link

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.

5 participants