Skip to content

[Forwardport] Wrong Last orders amount on dashboard #15660#15682

Merged
magento-engcom-team merged 7 commits intomagento:2.3-developfrom
ankurvr:ankurvr-patch-2.3-develop
Jun 2, 2018
Merged

[Forwardport] Wrong Last orders amount on dashboard #15660#15682
magento-engcom-team merged 7 commits intomagento:2.3-developfrom
ankurvr:ankurvr-patch-2.3-develop

Conversation

@ankurvr
Copy link
Copy Markdown
Member

@ankurvr ankurvr commented Jun 2, 2018

Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies #15660

Original Pull Request

#15661

Fixed Issues (if relevant)

  1. Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies #15660: Wrong order amount on dashboard on Last orders listing when having more than one website with different currencies

Manual testing scenarios

  1. Create 2 Websites with 1 store in each
  2. Set Default website/store base currency as INR (Indian Rupee)
  3. Set another website/store base currency as USD (US Dollar)
  4. Place Order from both websites with with their default currencies.
  5. Now go to Dashboard and set Store View as All Store Views
  6. See Last Orders table and check order's total amounts and all mounts will be in INR (USD orders will be converted to INR with exchange rate
  7. Now change Store View to USD currency website's store which have orders
  8. See Last Orders table and check order's total amounts. You will find those amounts converted to again into INR to USD. But at this point orders are already in USD so no need to convert amounts

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@magento-engcom-team
Copy link
Copy Markdown
Contributor

Hi @ankurvr. Thank you for your contribution.
Changes from your Pull Request will be available with the upcoming 2.3.0 release.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants