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

Failed to move to commerce when clicking Go To OMS when having oms url in specific pattern #275

Closed
1 of 2 tasks
ymaheshwari1 opened this issue Mar 18, 2024 · 3 comments
Closed
1 of 2 tasks

Failed to move to commerce when clicking Go To OMS when having oms url in specific pattern #275

ymaheshwari1 opened this issue Mar 18, 2024 · 3 comments
Assignees
@Adameddy26
Labels
bug Something isn't working

Comments

@ymaheshwari1
Copy link
Contributor

Current behavior

When clicking Go To OMS, the user does not move to the commerce

Expected behavior

The redirection to commerce should always happen to have a consistent user experience

Steps to reproduce the issue

  1. Go to commerce
  2. Click on Go To Lauunchpad
  3. Once logged-in into the launchpad, click the redirect icon next to oms name
  4. Verify that the commerce does not open

This can also regenerate when login into launchpad using oms value as https://<oms>.hotwax.io

Can you handle fixing this bug by yourself?

  • YES
  • NO

Environment details

  • Browser:
  • OS:
  • Code Version:

Additional information
@ymaheshwari1 ymaheshwari1 added the bug Something isn't working label Mar 18, 2024
@ymaheshwari1 ymaheshwari1 moved this from 📋 Backlog to 📅 Planned in Digital Experience Platform Mar 18, 2024
@ymaheshwari1 ymaheshwari1 self-assigned this Mar 18, 2024
@ymaheshwari1 ymaheshwari1 moved this from 📅 Planned to 🏗 In progress in Digital Experience Platform Mar 18, 2024
ymaheshwari1 added a commit to ymaheshwari1/ionic-sdk that referenced this issue Mar 18, 2024
@ymaheshwari1
Fixed: issue in redirection using GoToOms button by updating the rege…
775ab06 
…x for generating the link(hotwax#275)
@ymaheshwari1 ymaheshwari1 moved this from 🏗 In progress to 👀 In review in Digital Experience Platform Mar 18, 2024
@ymaheshwari1 ymaheshwari1 mentioned this issue Mar 18, 2024
Merged
3 tasks
ravilodhi added a commit that referenced this issue Mar 18, 2024
@ravilodhi
Merge pull request #276 from ymaheshwari1/#275
d0a8530 
Fixed: issue in redirection using GoToOms button by updating the regex for generating the link(#275)
@ymaheshwari1
Copy link
Contributor Author

Update Dxp package version in all apps.

@Adameddy26 Adameddy26 moved this from 👀 In review to 🛠 In QA in Digital Experience Platform Mar 18, 2024
@Adameddy26 Adameddy26 moved this from 🛠 In QA to ✅ Done in Digital Experience Platform Mar 18, 2024
@R-Sourabh R-Sourabh mentioned this issue Mar 18, 2024
Merged
1 task
@Adameddy26
Copy link

Issue verified and working fine in v1.17.0.
https://jam.dev/c/495a52b4-bfb7-4dac-b269-971dac8d17ee

This was referenced Mar 18, 2024
Closed
Merged
Closed
Closed
Closed
Merged
Closed
Merged
Merged
@ymaheshwari1 ymaheshwari1 mentioned this issue Mar 26, 2024
Merged
1 task
@Adameddy26
Copy link

Verified and working fine in facilities app release v1.5.0.

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

@Adameddy26 Adameddy26

Labels
bug Something isn't working
Projects
Digital Experience Platform
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ymaheshwari1 @Adameddy26