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

BOPIS (Buy Online, Pick Up In-Store) #274

Closed
Tracked by #124
Dhiraj1405 opened this issue Sep 8, 2023 · 4 comments · Fixed by #279
Closed
Tracked by #124

BOPIS (Buy Online, Pick Up In-Store) #274

Dhiraj1405 opened this issue Sep 8, 2023 · 4 comments · Fixed by #279
Assignees
@amansinghbais
Labels
enhancement New feature or request

Comments

@Dhiraj1405
Copy link
Contributor

No description provided.

@Dhiraj1405 Dhiraj1405 mentioned this issue Sep 8, 2023
Open
4 tasks
@Dhiraj1405 Dhiraj1405 transferred this issue from hotwax/dxp-components Sep 8, 2023
@Dhiraj1405 Dhiraj1405 added the enhancement New feature or request label Sep 8, 2023
This was referenced Sep 13, 2023
Closed
Merged
@Dhiraj1405 Dhiraj1405 reopened this Oct 17, 2023
@Dhiraj1405
Copy link
Contributor Author

Reopened as some issues found.

Here is the list of issues:

  1. Notification preference not updated on the facility change
  2. Browser notifications are not consistent
  3. Notification speed is too slow
  4. Currently, when users click on a notification from another tab, however, users are being redirected to a notification page that does not display any notifications.
  5. Notifications not shown when user is in the app.

@Dhiraj1405 Dhiraj1405 assigned Dhiraj1405 and unassigned k2maan Nov 8, 2023
amansinghbais added a commit to amansinghbais/bopis that referenced this issue Feb 1, 2024
@amansinghbais
Fixed: notification pref not updating on facility change issue (hotwa…
698a778 
…x#274)
@amansinghbais amansinghbais mentioned this issue Feb 2, 2024
Merged
1 task
amansinghbais added a commit to amansinghbais/bopis that referenced this issue Feb 2, 2024
@amansinghbais
Improved: added loader to update notification preference action(hotwa…
4fbab17 
…x#274)
@amansinghbais amansinghbais mentioned this issue Feb 5, 2024
Merged
1 task
amansinghbais added a commit to amansinghbais/bopis that referenced this issue Feb 5, 2024
@amansinghbais
Updated: version for oms-api in package.json(hotwax#274)
36e613b
ravilodhi added a commit that referenced this issue Feb 5, 2024
@ravilodhi
Merge pull request #375 from amansinghbais/274-notificationIssue
132dbf4 
Fixed: notification issues in bopis (#274)
amansinghbais added a commit to amansinghbais/bopis that referenced this issue Feb 9, 2024
@amansinghbais
Improved: logic to show loader while logout to show during notificati…
b27a6de 
…on unsubscribe(hotwax#274)
amansinghbais added a commit to amansinghbais/bopis that referenced this issue Feb 9, 2024
@amansinghbais
Fixed: notfications not coming up after app refresh (hotwax#274)
70b9074
amansinghbais added a commit to amansinghbais/bopis that referenced this issue Jun 14, 2024
@amansinghbais
Reverted: unwanted changes (hotwax#274)
1634840
@Dhiraj1405
Copy link
Contributor Author

@amansinghbais I see some of the items are taken care of. Should I created a seperate ticket for the items left, or just close this out because everything is already done.

@amansinghbais
Copy link
Contributor

@Dhiraj1405 sir, Almost all of the issues mentioned in this ticket are resolved. The one's left are due to a single reason that notifications don't come after refreshing the app.

We can close this ticket and raise a different for that refreshing issue.

@Dhiraj1405
Copy link
Contributor Author

Closing this issue, we will create a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amansinghbais amansinghbais

Labels
enhancement New feature or request
Projects
Digital Experience Platform
Archived in project
Milestone
No milestone
3 participants
@k2maan @amansinghbais @Dhiraj1405