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.

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

issue #25003 fixed #25004

Open
wants to merge 2 commits into
base: 2.4-develop
Choose a base branch
from

Conversation

anujwebkul
Copy link
Contributor

@anujwebkul anujwebkul commented Oct 12, 2019

Wrong currency symbol in creditmemo_grid & sales_order_view > creditmemo grid for Adjustment Refund & Adjustment Fee columns

Description (*)

For Columns Adjustment Refund & Adjustment Fee in creditmemo_grid & sales_order_view > credit memo grid if the order is placed with the different currency.

Fixed Issues (if relevant)

  1. Fixes Wrong currency symbol in creditmemo grid for Adjustment Refund & Adjustment Fee #25003 : Wrong currency symbol in creditmemo_grid & sales_order_view > credit memo grid for Adjustment Refund & Adjustment Fee Columns

Manual testing scenarios (*)

  1. Set USD as Base Currency
  2. Set INR currency for different store
  3. Place an order from INR Currency store.
  4. Go to Backend > SALES > Credit Memos, create Credit Memo with Adjustment Refund & Adjustment Fee
  5. Check Adjustment Refund & Adjustment Fee column on Credit Memo grid (sales_creditmemo_index) & sales_order_view > credit memo grid.

Questions or comments

Finally, Adjustment Refund & Adjustment Fee columns value is showing correctly currency symbol.

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 are green)

@anujwebkul anujwebkul requested a review from YevSent as a code owner October 12, 2019 09:10
@m2-assistant
Copy link

m2-assistant bot commented Oct 12, 2019

Hi @anujwebkul. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

  • @magento give me test instance - deploy test instance based on PR changes
  • @magento give me 2.3-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Guide documentation.

@anujwebkul
Copy link
Contributor Author

@joni-jones Please review this PR.

@rodrigowebjump rodrigowebjump self-requested a review October 28, 2019 17:59
@rodrigowebjump rodrigowebjump self-assigned this Oct 28, 2019
@rodrigowebjump
Copy link
Member

@magento give me test instance

@magento-engcom-team
Copy link
Contributor

Hi @rodrigowebjump. Thank you for your request. I'm working on Magento instance for you

@magento-engcom-team
Copy link
Contributor

Hi @rodrigowebjump, here is your new Magento instance.
Admin access: https://pr-25004.instances.magento-community.engineering/admin
Login: admin Password: 123123q

@sidolov sidolov changed the base branch from 2.3-develop to 2.4-develop December 5, 2019 17:18
@AleksLi
Copy link
Contributor

AleksLi commented Jun 11, 2020

I found where is the issue.
You simply can compare sales_invoice_grid and sales_creditmemo_grid column by column and you will see that there is no currency_code tables in creditmemo_grid table at all. But it should take the currency_code from there to show it in the grid.
I've found that \Magento\Sales\Api\Data\CreditmemoInterface::setBaseCurrencyCode has currency_code in their interface.

Also I've found out there is nothing in db_schema.xml for the creditmemo table.
My question is why? What that some kind of idea to make that in the other way?

But on version 2.3.5 of Magento 2 this is still an issue.

@sidolov sidolov added Priority: P3 May be fixed according to the position in the backlog. Severity: S3 Affects non-critical data or functionality and does not force users to employ a workaround. labels Aug 18, 2020
@orlangur orlangur self-assigned this Dec 20, 2020
@orlangur
Copy link
Contributor

@magento run all tests

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Frontend Component: Sales Event: cdwebkul Webkul Contribution Day Priority: P3 May be fixed according to the position in the backlog. Progress: review Release Line: 2.4 Severity: S3 Affects non-critical data or functionality and does not force users to employ a workaround. Squashtoberfest 2019
Projects
Status: Review in Progress
Development

Successfully merging this pull request may close these issues.

Wrong currency symbol in creditmemo grid for Adjustment Refund & Adjustment Fee
6 participants