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

Add ability to set tax config in Avalara plugin #10445

Conversation

mociepka
Copy link
Contributor

@mociepka mociepka commented Aug 21, 2022

I want to merge this change because I cannot set the "All products prices are entered with tax included." tax setting per channel.

This change extends Avalara plugin configuration with the ability to override the "All products prices are entered with tax included." per channel.

image

Impact

  • Documentation needs to be updated

Pull Request Checklist

  • Privileged queries and mutations are guarded by proper permission checks
  • Database queries are optimized and the number of queries is constant
  • Database migration files are up to date
  • The changes are tested
  • GraphQL schema and type definitions are up to date
  • Changes are mentioned in the changelog

saleor/plugins/avatax/plugin.py Show resolved Hide resolved
saleor/plugins/avatax/plugin.py Outdated Show resolved Hide resolved
@korycins korycins requested a review from a team August 22, 2022 07:26
Copy link
Member

@maarcingebala maarcingebala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good but it needs to things:

  • changelog
  • tests are currently failing

@mociepka mociepka force-pushed the add-apility-to-set-tax-config-in-avalara-plugin branch from 213bc3c to 36266a5 Compare August 22, 2022 10:08
@mociepka
Copy link
Contributor Author

@maarcingebala I missed one file in commit. I just pushed it. @maarcingebala, what and where do I need to do with the changelog?

@db-queries
Copy link

db-queries bot commented Aug 22, 2022

Here is the report for 2b5abb5 (mociepka:add-apility-to-set-tax-config-in-avalara-plugin)
Base comparison is 2052841.

No differences were found.

# saleor.graphql.accountbenchmark account
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  addresses for federation query count                                    	          9	          9	              2
  customers query                                                         	         48	         48	              0
  delete staff members                                                    	         37	         37	              1
  query staff user                                                        	         18	         18	              3
  staff create                                                            	         25	         25	              3
  staff update groups and permissions                                     	         36	         36	              5
  users for federation query count                                        	          8	          8	              3

# saleor.graphql.accountbenchmark permission group
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  groups for federation query count                                       	          5	          5	              2
  permission group create                                                 	         21	         21	              2
  permission group delete                                                 	         22	         22	              3
  permission group query                                                  	          7	          7	              0
  permission group update                                                 	         35	         35	              1
  permission group update remove users with manage staff                  	         29	         29	              3

# saleor.graphql.appbenchmarks app extensions
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  app extensions                                                          	         15	         15	              0
  app extensions with filter[filter0]                                     	         14	         14	              0
  app extensions with filter[filter1]                                     	         10	         10	              0
  app extensions with filter[filter2]                                     	         14	         14	              0
  app extensions with filter[filter3]                                     	         10	         10	              0

# saleor.graphql.appbenchmarks apps
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  apps for federation query count                                         	          9	          9	              3

# saleor.graphql.attributebenchmark attribute
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  attribute translation                                                   	          6	          6	              0
  attribute value translation                                             	         26	         26	              0
  query attribute                                                         	          6	          6	              0
  query attributes                                                        	          8	          8	              0

# saleor.graphql.channelbenchmark channel
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  channels query                                                          	          5	          5	              0

# saleor.graphql.channeltest channel update mutations
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  channel update mutation remove warehouse                                	         35	         35	              6

# saleor.graphql.checkoutbenchmark checkout mutations
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  add billing address to checkout                                         	         54	         54	              5
  add checkout lines                                                      	         54	         54	              4
  add checkout lines with external shipping                               	        151	        151	             55
  add checkout lines with reservations                                    	        114	        114	             30
  add delivery to checkout                                                	         52	         52	              5
  add shipping to checkout                                                	         61	         61	              8
  checkout email update                                                   	         27	         27	              0
  checkout payment charge                                                 	         36	         36	             12
  checkout shipping address update                                        	         63	         63	              7
  checkout voucher code                                                   	         70	         70	              9
  complete checkout                                                       	        144	        144	             22
  complete checkout preorder                                              	        158	        158	             31
  complete checkout with digital line                                     	        204	        204	             48
  complete checkout with out of stock webhook                             	        146	        146	             22
  complete checkout with single line                                      	        146	        146	             22
  create checkout                                                         	         70	         70	             10
  create checkout with reservations                                       	        122	        122	             20
  customer complete checkout                                              	        159	        159	             26
  customer complete checkout for cc                                       	        147	        147	             22
  update checkout lines                                                   	         47	         47	              4
  update checkout lines with reservations                                 	        120	        120	             39

# saleor.graphql.checkoutbenchmark homepage
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  user checkout details                                                   	         43	         43	              2
  user checkout details with tax app                                      	         42	         42	              1

# saleor.graphql.discountbenchmark sales
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  sales query with channel slug                                           	         19	         19	              0
  sales query withot channel slug                                         	         18	         18	              0

# saleor.graphql.discountbenchmark vouchers
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  vouchers query with channel slug                                        	         19	         19	              0
  vouchers query withot channel slug                                      	         18	         18	              0

# saleor.graphql.giftcardbenchmark gift card mutations
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  create never expiry gift card                                           	         28	         28	              4
  gift card bulk activate by staff                                        	         16	         16	              3
  update gift card                                                        	         36	         36	              4

# saleor.graphql.giftcardbenchmark gift card queries
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  filter gift cards by products                                           	         14	         14	              3
  filter gift cards by tags                                               	         12	         12	              3
  filter gift cards by used by user                                       	         13	         13	              3
  query gift card details                                                 	         14	         14	              3
  query gift cards                                                        	         12	         12	              3

# saleor.graphql.orderbenchmark fulfillment
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  fulfillment query                                                       	          9	          9	              0

# saleor.graphql.orderbenchmark fulfillment refund and return products
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  fulfillment refund products order lines                                 	         56	         56	              2
  fulfillment return products order lines                                 	        100	        100	              9

# saleor.graphql.orderbenchmark order
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  staff multiple draft orders                                             	        614	        614	            472
  staff multiple orders                                                   	        164	        164	             87
  staff order details                                                     	         48	         48	              8
  user order details                                                      	         46	         46	              9

# saleor.graphql.orderbenchmark order fulfill
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  order fulfill                                                           	         19	         19	              3
  order fulfill with gift cards                                           	         19	         19	              3

# saleor.graphql.pagebenchmark page type
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  page types for federation query count                                   	          3	          3	              0
  query page type                                                         	         18	         18	              6
  query page types                                                        	         25	         25	             12

# saleor.graphql.paymentbenchmark payment transactions
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  payment transactions                                                    	         49	         49	             20

# saleor.graphql.productbenchmark category
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  categories children                                                     	          3	          3	              0
  categories for federation query count                                   	          3	          3	              0
  category delete                                                         	         82	         82	             17
  category view                                                           	         27	         27	              1

# saleor.graphql.productbenchmark collection
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  collection add products                                                 	         36	         36	              6
  collection bulk delete                                                  	         48	         48	              8
  collection view                                                         	          6	          6	              0
  collections for federation query count                                  	          7	          7	              2
  create collection                                                       	         36	         36	              5
  delete collection                                                       	         40	         40	              7
  remove products from collection                                         	         33	         33	              6
  retrieve collection channel listings                                    	          5	          5	              0

# saleor.graphql.productbenchmark homepage
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  report product sales                                                    	         10	         10	              3
  retrieve product list                                                   	          4	          4	              0

# saleor.graphql.productbenchmark product
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  filter products by attributes                                           	          9	          9	              0
  filter products by boolean attributes                                   	         19	         19	              0
  filter products by gift card                                            	         11	         11	              1
  filter products by numeric attributes                                   	         18	         18	              0
  product create                                                          	         73	         73	              8
  product details                                                         	         35	         35	              0
  product translations                                                    	          5	          5	              0
  products for federation query count                                     	          6	          6	              2
  products media for federation query count                               	          3	          3	              0
  products types for federation query count                               	          2	          2	              0
  retrieve channel listings                                               	         17	         17	              0
  retrieve product attributes                                             	          9	          9	              0
  retrieve product images                                                 	          4	          4	              0
  retrieve product media                                                  	          4	          4	              0
  retrive products with product types and attributes                      	          7	          7	              0
  update product                                                          	         51	         51	             11

# saleor.graphql.productbenchmark product variant channel listing update
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  variant channel listing update                                          	         45	         45	              3

# saleor.graphql.productbenchmark variant
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  product variant bulk create                                             	         66	         66	              3
  product variant create                                                  	         80	         80	              5
  products variants for federation query count                            	          6	          6	              2
  retrieve variant list                                                   	         30	         30	              1
  update product variant                                                  	         78	         78	              9

# saleor.graphql.productbenchmark variant stocks
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  product variants stocks create                                          	         26	         26	              4
  product variants stocks create with single webhook called               	         25	         25	              4
  product variants stocks delete                                          	         27	         27	              4
  product variants stocks delete with out of stock webhook many calls     	         23	         23	              3
  product variants stocks update                                          	         28	         28	              4
  query product variants stocks                                           	         10	         10	              0

# saleor.graphql.producttest product sorting attributes
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  sort product not having attribute data                                  	         23	         23	              0

# saleor.graphql.shippingbenchmark shipping methods
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  vouchers query with channel slug                                        	          9	          9	              0
  vouchers query without channel slug                                     	          9	          9	              0

# saleor.graphql.shopbenchmark homepage
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  retrieve shop                                                           	          4	          4	              0

# saleor.graphql.warehousebenchmark stocks
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  stocks query                                                            	          5	          5	              0

# saleor.graphql.webhookbenchmark webhook events
  test name                                                               	left count 	right count	duplicate count
  ------------------------------------------------------------------------	-----------	-----------	---------------
  webhooks                                                                	          9	          9	              0

@codecov
Copy link

codecov bot commented Aug 22, 2022

Codecov Report

Merging #10445 (2b5abb5) into main (2052841) will increase coverage by 0.00%.
The diff coverage is 90.00%.

@@           Coverage Diff           @@
##             main   #10445   +/-   ##
=======================================
  Coverage   93.05%   93.06%           
=======================================
  Files         632      632           
  Lines       49773    49767    -6     
  Branches     8333     8325    -8     
=======================================
- Hits        46318    46317    -1     
  Misses       2257     2257           
+ Partials     1198     1193    -5     
Impacted Files Coverage Δ
saleor/plugins/avatax/tests/conftest.py 100.00% <ø> (ø)
saleor/plugins/avatax/plugin.py 74.10% <81.81%> (+0.86%) ⬆️
saleor/plugins/avatax/__init__.py 93.79% <100.00%> (+0.02%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@maarcingebala
Copy link
Member

@patrys Are we going to release these changes as 3.7? I would say so as this is new feature. If yes, then the changelog should be added to the 3.7.0 [Unreleased] section .

@patrys
Copy link
Member

patrys commented Aug 22, 2022

@maarcingebala This can't wait for taxes so either 3.6 or we release 3.7 now and move taxes to 3.8.

@mociepka
Copy link
Contributor Author

@maarcingebala @patrys I extended the 3.6 change log with information about this change.

@timuric
Copy link
Member

timuric commented Aug 23, 2022

@mociepka could you explain what "Override global tax settings" means.
Does it override all tax settings?
image

CHANGELOG.md Outdated Show resolved Hide resolved
@mociepka mociepka force-pushed the add-apility-to-set-tax-config-in-avalara-plugin branch from c84e058 to 98fb5c7 Compare August 23, 2022 15:15
@mociepka
Copy link
Contributor Author

@timuric override setting listed below. Only one is listed:
image

image

I can change wording no problem with that, but please provide labels and tool tips.

@mociepka mociepka force-pushed the add-apility-to-set-tax-config-in-avalara-plugin branch 2 times, most recently from 98fb5c7 to 3b9e6f1 Compare August 23, 2022 16:40
@mociepka mociepka force-pushed the add-apility-to-set-tax-config-in-avalara-plugin branch from 3b9e6f1 to aa3e527 Compare August 23, 2022 17:04
@mociepka mociepka force-pushed the add-apility-to-set-tax-config-in-avalara-plugin branch from cef1416 to 2b5abb5 Compare August 23, 2022 17:19
@maarcingebala maarcingebala merged commit eb29c07 into saleor:main Aug 24, 2022
@mociepka mociepka deleted the add-apility-to-set-tax-config-in-avalara-plugin branch August 24, 2022 07:10
@maarcingebala maarcingebala added this to the Saleor 3.7 milestone Aug 24, 2022
maarcingebala pushed a commit that referenced this pull request Aug 26, 2022
* Add tax override settings to the Avalara plugin

* Use explicit tax_included arg for avalara methods

* Fix existing avalara tests

* Add tests for avalara override settings plugin

* Move tax_included to plugin configuration

* Update change log
maarcingebala added a commit that referenced this pull request Aug 29, 2022
* Add ability to set tax config in Avalara plugin (#10445)

* Add tax override settings to the Avalara plugin

* Use explicit tax_included arg for avalara methods

* Fix existing avalara tests

* Add tests for avalara override settings plugin

* Move tax_included to plugin configuration

* Update change log

* Update changelog

Co-authored-by: Michał Ociepka <michal@mirumee.com>
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

5 participants