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

Test - Adresses #72

Open
3 tasks done
georgmaisser opened this issue Jun 2, 2024 · 4 comments
Open
3 tasks done

Test - Adresses #72

georgmaisser opened this issue Jun 2, 2024 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@georgmaisser
Copy link
Contributor

georgmaisser commented Jun 2, 2024

Address is important for taxes. We have billing & shipping Address. Only billing Address should have effect on taxes etc.

  • - Test if different countries get the right VAT according to their respective tax categories.

  • - Test if different mode of net prices (added or included, depending on shopping cart settings) work as expected

  • - don't test the address form for the moment, just the result. We might remodel the form to a dynamic form. Implementation at the moment is old and not according to Moodle APIs

@semteacher semteacher added the enhancement New feature or request label Jun 3, 2024
@eynimeni
Copy link
Contributor

eynimeni commented Jun 3, 2024

see link to screen recordings with details in #71

@semteacher
Copy link
Collaborator

semteacher commented Jun 3, 2024

image

semteacher added a commit that referenced this issue Jun 4, 2024
semteacher added a commit that referenced this issue Jun 4, 2024
semteacher added a commit that referenced this issue Jun 4, 2024
semteacher added a commit that referenced this issue Jun 4, 2024
@semteacher semteacher mentioned this issue Jun 4, 2024
6 tasks
semteacher added a commit that referenced this issue Jun 4, 2024
…w behat Scenario: Shopping Cart taxes: use VAT number and German address to reduce gross price of single item (#71 #72)
@georgmaisser
Copy link
Contributor Author

Hi Andrii, we didn't refactor the address yet, so no need for further work at the moment. the js issue is not there anymore, I presume?

@semteacher
Copy link
Collaborator

@georgmaisser yes, JS error was solved, tests for 3 of 3 cases also alrady existed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

No branches or pull requests

3 participants