Skip to content

Conversation

@malinajirka
Copy link
Contributor

@malinajirka malinajirka commented Sep 9, 2025

Step 7 needs to be merged first.

Description

This is part of the migration to POS specific Product Model. There will be many changes and I want to avoid huge PRs. Therefore I created a feature branch - I'll create a PR for each set of changes. The CI won't be able to build this PR, but I personally wouldn't worry about it -> we can ensure everything is green and working as expected when we are merging the feature branch back to trunk.

The Version2 in the name WooPosProductModelVersion2 will eventually be removed.

Images/gif

N/A

  • I have considered if this change warrants release notes and have added them to RELEASE-NOTES.txt if necessary. Use the "[Internal]" label for non-user-facing changes.

@malinajirka malinajirka changed the title POS Product Migration Step 8 - final step POS Product Migration Step 8 Sep 9, 2025
@malinajirka malinajirka added this to the 23.3 milestone Sep 9, 2025
@wpmobilebot
Copy link
Collaborator

📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.
App Name WooCommerce-Wear Android
Platform⌚️ Wear OS
FlavorJalapeno
Build TypeDebug
Commit0ae7ac6
Direct Downloadwoocommerce-wear-prototype-build-pr14593-0ae7ac6.apk

@malinajirka malinajirka requested a review from kidinov September 9, 2025 07:20
@malinajirka malinajirka added the status: do not merge Dependent on another PR, ready for review but not ready for merge. label Sep 9, 2025
@dangermattic
Copy link
Collaborator

1 Error
🚫 This PR is tagged with status: do not merge label(s).

Generated by 🚫 Danger

@wpmobilebot
Copy link
Collaborator

📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.

App Name WooCommerce Android
Platform📱 Mobile
FlavorJalapeno
Build TypeDebug
Commit0ae7ac6
Direct Downloadwoocommerce-prototype-build-pr14593-0ae7ac6.apk

@kidinov kidinov self-assigned this Sep 9, 2025
Copy link
Contributor

@kidinov kidinov left a comment

Choose a reason for hiding this comment

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

LGTM!

Base automatically changed from issue/woomob-1280-woo-poslocal-catalog-use-new-pos-specific-product-model-step-7 to feature-branch-for-pos-product-migration September 10, 2025 12:49
@malinajirka malinajirka merged commit c65faf1 into feature-branch-for-pos-product-migration Sep 10, 2025
18 of 23 checks passed
@malinajirka malinajirka deleted the issue/woomob-1280-woo-poslocal-catalog-use-new-pos-specific-product-model-step-8 branch September 10, 2025 12:49
@malinajirka malinajirka mentioned this pull request Sep 10, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: POS status: do not merge Dependent on another PR, ready for review but not ready for merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants