Skip to content

Commit

Permalink
Stop using code which resolves default display tax label value per co…
Browse files Browse the repository at this point in the history
…untry and segment (#27)

Co-authored-by: Bozo Jovicic <bozo@hitthecode.com>
Co-authored-by: Ilyas Türkben <isturkben@gmail.com>
Co-authored-by: Mariia Lukianets <lukianet@adobe.com>
  • Loading branch information
4 people authored Jul 23, 2024
1 parent 339d1c5 commit c9d55dc
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
4 changes: 4 additions & 0 deletions commerce/src/inline-price.js
Original file line number Diff line number Diff line change
Expand Up @@ -222,11 +222,15 @@ export class HTMLPriceSpanElement extends HTMLSpanElement {
);
if (!options.wcsOsi.length) return false;

/*
Commented out until issues in content with manually added tax labels are resolved
if (!this.placeholder.dataset.displayTax) {
// set default value for displayTax if not set neither in OST nor in price URL
options.displayTax =
(await this.resolveDisplayTax(service, options)) || false;
}
*/

const version = this.placeholder.togglePending(options);
this.innerHTML = '';
Expand Down
4 changes: 4 additions & 0 deletions commerce/test/price.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -288,6 +288,9 @@ describe('class "InlinePrice"', () => {
});
});

/*
Commented out until issues in content with manually added tax labels are resolved
describe('default display tax', () => {
const SEGMENTS = ['individual', 'business', 'student', 'university'];
const TESTS = [
Expand Down Expand Up @@ -676,6 +679,7 @@ describe('class "InlinePrice"', () => {
expect(inlinePrice.textContent).equal('US$89.99/mo');
});
});
*/
});

describe('commerce service', () => {
Expand Down

0 comments on commit c9d55dc

Please sign in to comment.