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

fix: Taxjar customer_address fix, currency fix #28262

Conversation

nemesis189
Copy link
Contributor

@nemesis189 nemesis189 commented Nov 8, 2021

Problems:

  • customer_address_name was wrong variable, corrected it to customer_address
  • Sales invoice item table had fields showing wrong currency, added currency to 'options' of Tax Collectable and Taxable Amount fields
  • In case of multiple companies, make api call only if invoice is from US

@nemesis189 nemesis189 requested review from deepeshgarg007 and removed request for nextchamp-saqib November 8, 2021 04:27
@nemesis189 nemesis189 changed the title Taxjar customer addressfix, currency fix Taxjar customer_address fix, currency fix Nov 8, 2021
@deepeshgarg007 deepeshgarg007 changed the title Taxjar customer_address fix, currency fix fix: Taxjar customer_address fix, currency fix Nov 8, 2021
@deepeshgarg007
Copy link
Member

@Mergifyio backport develop

@deepeshgarg007
Copy link
Member

@Mergifyio backport version-13-pre-release

@mergify
Copy link
Contributor

mergify bot commented Nov 8, 2021

backport develop

🟠 Waiting for conditions to match

  • merged [:pushpin: backport requirement]

@mergify
Copy link
Contributor

mergify bot commented Nov 8, 2021

backport version-13-pre-release

🟠 Waiting for conditions to match

  • merged [:pushpin: backport requirement]

@deepeshgarg007 deepeshgarg007 merged commit bad6c3d into frappe:version-13-hotfix Nov 8, 2021
@mergify
Copy link
Contributor

mergify bot commented Nov 8, 2021

backport version-13-pre-release

✅ Backports have been created

@mergify
Copy link
Contributor

mergify bot commented Nov 8, 2021

backport develop

✅ Backports have been created

deepeshgarg007 added a commit that referenced this pull request Nov 8, 2021
…se/pr-28262

fix: Taxjar customer_address fix, currency fix (backport #28262)
deepeshgarg007 added a commit that referenced this pull request Nov 8, 2021
fix: Taxjar customer_address fix, currency fix (backport #28262)
asoral pushed a commit to asoral/erpnext that referenced this pull request Nov 12, 2021
…-release/pr-28262

fix: Taxjar customer_address fix, currency fix (backport frappe#28262)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants