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

When using oms complete url to login, the redirection to commerce from inventory detail and PO detail page fails #268

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

When using oms complete url to login, the redirection to commerce from inventory detail and PO detail page fails #268

ymaheshwari1 opened this issue Mar 18, 2024 · 1 comment
Assignees
@Adameddy26
Labels
bug Something isn't working release-2.15.0 Release 2.15.0

Comments

@ymaheshwari1
Copy link
Contributor

Current behavior

After successfully uploading an inventory or PO file, a toast gets displayed to view the uploaded file in commerce, that does not work when the user is logged-in into the app using complete URL https://<oms>.hotwax.io/api.

Expected behavior

The redirection should always work irrespective of the oms pattern, user used to logged-in into the app.

Steps to reproduce the issue

  1. Log into the app using https://<oms>.hotwax.io/api as the oms name
  2. Upload inventory or PO file
  3. Click on the View button in the toast to get redirected to the commerce and verify that the page does not open

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 self-assigned this Mar 18, 2024
@ymaheshwari1 ymaheshwari1 moved this from 📋 Backlog to 📅 Planned in Digital Experience Platform Mar 18, 2024
ymaheshwari1 added a commit to ymaheshwari1/import that referenced this issue Mar 18, 2024
@ymaheshwari1
Fixed: issue when redirecting to commerce after successfull upload of…
82acf5c 
… inventory or PO file(hotwax#268)
@ymaheshwari1 ymaheshwari1 mentioned this issue Mar 18, 2024
Merged
1 task
@ymaheshwari1 ymaheshwari1 moved this from 📅 Planned to 🏗 In progress in Digital Experience Platform Mar 18, 2024
ymaheshwari1 added a commit to ymaheshwari1/import that referenced this issue Mar 18, 2024
@ymaheshwari1
Improved: regex to remove duplicate / in some cases(hotwax#268)
ce2d6c9
ymaheshwari1 added a commit that referenced this issue Mar 18, 2024
@ymaheshwari1
Merge pull request #269 from ymaheshwari1/#268
b740b23 
Fixed: issue when redirecting to commerce after successfull upload of inventory or PO file(#268)
@ymaheshwari1 ymaheshwari1 moved this from 🏗 In progress to 👀 In review in Digital Experience Platform Mar 18, 2024
@ymaheshwari1 ymaheshwari1 moved this from 👀 In review to ⏸️ Dev complete in Digital Experience Platform Mar 18, 2024
@Adameddy26 Adameddy26 added the release-2.15.0 Release 2.15.0 label Mar 19, 2024
@Adameddy26 Adameddy26 moved this from ⏸️ Dev complete to 🛠 In QA in Digital Experience Platform Mar 19, 2024
@Adameddy26 Adameddy26 moved this from 🛠 In QA to ✅ Done in Digital Experience Platform Mar 19, 2024
@Adameddy26
Copy link

Issue verified and working fine in v2.15.0.
https://jam.dev/c/1520b71c-4c5a-438b-b211-590afeae4f97

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 release-2.15.0 Release 2.15.0
Projects
Digital Experience Platform
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ymaheshwari1 @Adameddy26