Skip to content

Conversation

@JoaoPinhoMinder
Copy link
Contributor

@JoaoPinhoMinder JoaoPinhoMinder commented Mar 13, 2025

Ticket

https://mindera.atlassian.net/browse/ALFMOB-158

Depends on: #42

Description

  • Added productsPublisher to both WishlistService and BagService to enable real-time updates of button titles based on item state.
  • Updated all relevant view models to observe productsPublisher instead of using the previous getWishlistContent and getBagContent methods.
  • Adjusted button titles dynamically to reflect the correct state ("Add to Bag" / "Remove from Bag" and "Add to Wishlist" / "Remove from Wishlist").
  • Ensured that changing color resets both buttons, while changing size only resets the "Add to Bag" button (since wishlist state is unaffected by size).
  • Added necessary localized strings to L10n table.

Evidences

Screen.Recording.2025-03-13.at.13.21.27.mov

@JoaoPinhoMinder JoaoPinhoMinder force-pushed the task/ALFMOB-158-update-pdp-button-copy-based-on-item-state branch from f5d495f to 6b6da16 Compare March 13, 2025 14:38
@JoaoPinhoMinder JoaoPinhoMinder marked this pull request as ready for review March 13, 2025 14:38
@p4checo
Copy link
Contributor

p4checo commented Mar 25, 2025

This PR seems to build on top of #42, so ideally we should have that not on the PR's description, and point it to the #42 PR which then creates a merge train 🚋

@JoaoPinhoMinder JoaoPinhoMinder marked this pull request as draft March 26, 2025 12:25
@JoaoPinhoMinder JoaoPinhoMinder force-pushed the task/ALFMOB-158-update-pdp-button-copy-based-on-item-state branch from 6b6da16 to 6f67ef7 Compare April 2, 2025 13:18
@JoaoPinhoMinder JoaoPinhoMinder changed the base branch from main to task/ALFMOB-162-improve-wishlist-functionality-on-plp-and-wishlist April 2, 2025 13:19
@JoaoPinhoMinder JoaoPinhoMinder force-pushed the task/ALFMOB-162-improve-wishlist-functionality-on-plp-and-wishlist branch from ae00fcf to 18bc8b0 Compare May 5, 2025 16:16
- Added `productsPublisher` to both `WishlistService` and `BagService` to enable real-time updates of button titles based on item state.
- Updated all relevant view models to observe `productsPublisher` instead of using the previous `getWishlistContent` and `getBagContent` methods.
- Adjusted button titles dynamically to reflect the correct state (`"Add to Bag"` / `"Remove from Bag"` and `"Add to Wishlist"` / `"Remove from Wishlist"`).
- Ensured that changing color resets both buttons, while changing size only resets the `"Add to Bag"` button (since wishlist state is unaffected by size).
- Added necessary localized strings to `L10n` table.
@JoaoPinhoMinder JoaoPinhoMinder force-pushed the task/ALFMOB-158-update-pdp-button-copy-based-on-item-state branch from 6f67ef7 to 060e707 Compare May 5, 2025 16:53
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.

3 participants