[BB-4552] feat: Add support for fixed rate in ecommerce

[alfredchavez]

BB-4552: Add support for a fixed rate for all products in ecommerce

Description

Ecommerce does not support tax rates for its payments(DefaultStrategy uses a NoTax strategy), this PR adds support for a fixed tax rate through a new django setting TAX_RATE that is applied to all products. Also the default value for the TAX_RATE setting is 0.0, preserving the previous platform behaviour.

Supporting information

Jira Issue: BB-4552

Testing instructions

• Check that the added tests make sense and run without failure

• Change the TAX_RATE to a decimal value(like "0.3")

• Try to make a purchase(could be the certificate upgrade of the demo course)

• Verify that the total value displayed on basket and invoice includes the tax rate

[openedx-webhooks]

Thanks for the pull request, @alfredchavez! I've created OSPR-5961 to keep track of it in JIRA, where we prioritize reviews. Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

[giovannicimolin]

@natabene @nedbat It looks like the CI checks weren't triggered for this (@alfredchavez is a member of OpenCraft, and this was created from our fork). Can you check or trigger a CI run for this?

[giovannicimolin]

~~jenkins run all~~

Didn't work, but it was worth a shot...

[natabene]

@alfredchavez Thank you for your contribution. @giovannicimolin I will kick them off now.

[giovannicimolin]

@alfredchavez Quality checks are failing, can you fix them (plus any other CI issues) and ping me for another review round Monday?

[alfredchavez]

Hi @giovannicimolin I have fixed the "import order" issues, but CI checks are not running until approval

[giovannicimolin]

@natabene Can you kick off the CI checks here again? Is there something we can do to unblock these checks from OpenCraft repos?

[giovannicimolin]

@alfredchavez Can you double-check that all tests are passing locally?

[giovannicimolin]

@natabene Can you trigger a CI run for this again?

[natabene]

Ok, I think they are running now.

[mphilbrick211]

@giovannicimolin is this still in progress? Or, can it be closed?

[mphilbrick211]

@giovannicimolin is this still in progress? Or, can it be closed?

Friendly ping on this @giovannicimolin :)

[giovannicimolin]

@mphilbrick211 Sorry for missing the pings.

We will not be pursuing this any longer, but I don't have permission to close the PR. Can you close it for me?

[mphilbrick211]

Closing, per Giovanni.

[openedx-webhooks]

@alfredchavez Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.