Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Use product identifier plugin #250

Closed
dt2patel opened this issue Jul 5, 2023 · 1 comment · Fixed by #337
Closed

Use product identifier plugin #250

dt2patel opened this issue Jul 5, 2023 · 1 comment · Fixed by #337
Assignees
@ymaheshwari1
Labels
enhancement New feature or request

Comments

@dt2patel
Copy link
Contributor

dt2patel commented Jul 5, 2023

What is the motivation for adding/enhancing this feature?

The BOPIS screen shows many product identifiers but many are redundant creating a cluttered experience.

What are the acceptance criteria?

The user should able to select their primary and secondary products identifier. Implemented using the DXP component for selecting product identifiers.
@dt2patel dt2patel added the enhancement New feature or request label Jul 5, 2023
@alsoK2maan alsoK2maan assigned sanskar345 and unassigned alsoK2maan Jul 13, 2023
@sanskar345 sanskar345 mentioned this issue Jul 17, 2023
Closed
1 task
sanskar345 added a commit to sanskar345/bopis that referenced this issue Aug 18, 2023
@sanskar345
Merge branch 'main' of https://github.com/hotwax/bopis into bopis/hot…
ab6c184 
…wax#250
sanskar345 added a commit to sanskar345/bopis that referenced this issue Aug 22, 2023
@sanskar345
Implemented: show toast to user if failed to update product identific…
229a61b 
…ation, remove catch in getProductIdentification, and fetch product identification if facility changes (hotwax#250)
sanskar345 added a commit to sanskar345/bopis that referenced this issue Aug 22, 2023
@sanskar345
Improved: locales file internationalization text order (hotwax#250)
1777c4e
sanskar345 added a commit to sanskar345/bopis that referenced this issue Aug 24, 2023
@sanskar345
Implemented: handling if api failed to update product identification …
dbe64ab 
…pref (hotwax#250)
sanskar345 added a commit to sanskar345/bopis that referenced this issue Aug 24, 2023
@sanskar345
Implemented: identifier updation failure message in locales file (hot…
8180b6c 
…wax#250)
@dt2patel
Copy link
Contributor Author

moving to planned status since linked pr is closed

ymaheshwari1 added a commit to ymaheshwari1/ionic-bopis that referenced this issue Nov 6, 2023
@ymaheshwari1
Implemented: support to use the product identifier component and used…
b62acd2 
… primary and secondary identifier accordingly(hotwax#250)
@ymaheshwari1 ymaheshwari1 mentioned this issue Nov 6, 2023
Merged
1 task
ymaheshwari1 added a commit to ymaheshwari1/ionic-bopis that referenced this issue Nov 7, 2023
@ymaheshwari1
Improved: logic to remove color and size from UI and used primary and…
3fe7953 
… secondary identifier(hotwax#250)
ymaheshwari1 added a commit to ymaheshwari1/ionic-bopis that referenced this issue Nov 7, 2023
@ymaheshwari1
Removed: unwanted label from the locales file(hotwax#250)
7589e9b
ravilodhi added a commit that referenced this issue Nov 7, 2023
@ravilodhi
Merge pull request #337 from ymaheshwari1/bopis/#250-updated
41cf0bb 
Implemented: support to use the product identifier component and used primary and secondary identifier accordingly(#250)
ymaheshwari1 added a commit to ymaheshwari1/ionic-bopis that referenced this issue Nov 8, 2023
@ymaheshwari1
Fixed: issue of product identifier api fails as token is not set befo…
1ee2301 
…re fetching product identifier(hotwax#250)
ravilodhi added a commit that referenced this issue Nov 8, 2023
@ravilodhi
Merge pull request #340 from ymaheshwari1/bopis/#250-updated
cfc780f 
Fixed: issue of product identifier api fails as token is not set before fetching product identifier(#250)
ymaheshwari1 added a commit to ymaheshwari1/ionic-bopis that referenced this issue Nov 8, 2023
@ymaheshwari1
Fixed: issue of product identifier api fails as token is not set befo…
5bf6d5e 
…re fetching product identifier(hotwax#250)
ravilodhi added a commit that referenced this issue Nov 8, 2023
@ravilodhi
Merge pull request #342 from ymaheshwari1/bopis/#250-updated
cf6d0e7 
Fixed: issue of product identifier api fails as token is not set before fetching product identifier(#250)
@ymaheshwari1 ymaheshwari1 mentioned this issue Nov 8, 2024
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ymaheshwari1 ymaheshwari1

Labels
enhancement New feature or request
Projects
Digital Experience Platform
Archived in project
Milestone
No milestone
4 participants
@ymaheshwari1 @alsoK2maan @dt2patel @sanskar345