Add more options for customizations and translations

[simonlerpard]

Adding multiple options to improve customizations and language adoptions,
as well as the currency position.

It's no breaking change, simply just more options 😃

[simonlerpard]

Also a lot of package updates during npm install. But that was just a bonus. There are still a lot of major versions needed to be updated etc.

[omahili]

A better alternative and naming for this property could be:

private currencyPosition: 'start' | 'end';

[omahili]

For consistency I would create a constant DEFAULT_CURRENCY_POSITION and set it to 'start', so this would be:

this.currencyPosition = cart?.currencyPosition || DEFAULT_CURRENCY_POSITION;

[omahili]

You can format the price once before creating the string:

const formattedValue = price.toFixed(2);
return this.currencyPosition === 'start'
  ? `${this.currency}${formattedValue}`
  : `${formattedValue}${this.currency}`;

[omahili]

Same improvements for currencyBehind as mentioned for CartItem.

[omahili]

Also here I would create formattedValue before building the string.

[omahili]

currencyLocation naming can be improved here to formattedCurrency.

[omahili]

showTotal was added but it's not used anywhere.

[omahili]

As mentioned above this would become:

currencyPosition: 'start' | 'end';

[omahili]

I see you probably used a newer version of npm and this has lockfileVersion 3. However I wouldn't update it in this PR, so it would be nice if you could revert it to original.

[omahili]

Hi @simonlerpard, thank you very much for contributing! I left a few comments here and there to improve namings and options structure.

[simonlerpard]

Awesome @omahili, I kind of did this in a rush... (without actually thinking it through all the way 😜)

I've just quickly looked at your comments and they are very reasonable. I'll look into this further soon. Hopefully during the weekend or the next. I'll update the PR again when it's done.

[omahili]

No worries @simonlerpard! For the labels I'm actually thinking if it makes more sense to have a full object in the options for locales like:

locales: {
  en: {
    front: 'Front',
    currency: '£',
  },
  it: {
    front: 'Fronte',
    currency: '€',
  }
}

And possibly a function to set locale like:

sc.setLocale('it')

So for now I removed all comments regarding labels. I will evaluate the solution mentioned here and eventually create another PR. I will let you know anyway!