Skip to content

Remove favoriteProducts feature flag #15786

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

Merged
merged 2 commits into from
Jun 23, 2025

Conversation

joshheald
Copy link
Contributor

@joshheald joshheald commented Jun 19, 2025

Description

This PR removes the favourite products feature flag, which was released a long time ago.

Steps to reproduce

  1. Launch the app and open the Products tab
  2. Check that you can still filter by favourite products
  3. Check that you can still mark a product as favourite (using the ... menu)

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@joshheald joshheald added this to the 22.7 milestone Jun 19, 2025
@joshheald joshheald requested a review from jaclync June 19, 2025 16:53
@joshheald joshheald added the type: task An internally driven task. label Jun 19, 2025
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Jun 19, 2025

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

App NameWooCommerce iOS Prototype
Build Number30665
VersionPR #15786
Bundle IDcom.automattic.alpha.woocommerce
Commit7c56b3b
Installation URL2ft1la1p80dhg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Copy link
Contributor

@jaclync jaclync left a comment

Choose a reason for hiding this comment

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

LGTM! TIL about the favorite products feature 😅 Some observations, I'm not sure if it's a feature we're actively maintaining though:

  • The CTA felt a bit hidden, I was expecting a heart or similar icon/CTA in the product details page like in most of the apps.
  • If the product list is filtered by favorite products only in a split view like on tablets, the favorite action from a unfavorite product before the filtering doesn't add the product to the list until clearing the filter and applying the favorite filter again.

@joshheald
Copy link
Contributor Author

LGTM! TIL about the favorite products feature 😅 Some observations, I'm not sure if it's a feature we're actively maintaining though:

  • The CTA felt a bit hidden, I was expecting a heart or similar icon/CTA in the product details page like in most of the apps.
  • If the product list is filtered by favorite products only in a split view like on tablets, the favorite action from a unfavorite product before the filtering doesn't add the product to the list until clearing the filter and applying the favorite filter again.

Thanks for the review 😊

I agree with you, but I don't think we're going to put time in to this just now. It's an old HACK week project from Sharma, only on iOS.

If we did more with it, the value should be synced to the products on the site.

@joshheald joshheald enabled auto-merge June 23, 2025 12:35
@joshheald joshheald merged commit 429a174 into trunk Jun 23, 2025
13 checks passed
@joshheald joshheald deleted the task/remove-favourite-products-flag branch June 23, 2025 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: task An internally driven task.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants