feat: adding new free shipping bar functionality

[carlosviniciusananias]

What is this contribution about?

Implementation of a free shipping bar in minicart addressing issue #112

Extra info

[carlosviniciusananias]

@lucis @guifromrio hey guys, could you look? sorry for mistakes, it's my first time working with deco.cx

[mcandeia]

Great job! Amazing!

Thanks for contributing!

[mcandeia]

since this component will likely be used by a huge amount of people, I think it may worth to add at least short descriptions here

e.g.: https://github.com/deco-sites/fashion/blob/main/components/header/Header.tsx#L34

[carlosviniciusananias]

@mcandeia ahh, yes... great suggestion

[mcandeia]

How about use a config key instead?

you can let the user input a sort of templatestring e.g

export interface Messages {
    leftTofreeShippingTemplate: string // "Faltam {currency} {value} para o frete grátis"
    freeShipping: string // "Parabéns! Você tem frete grátis"
}

export interface Props {
  value?: number;
  messages: Messages;
}

Then you can replace when using it

const leftToShipping = props.message.leftToShippingTemplate.replace("{currency}", currencyCode).replace("{value}", formatPrice(value - total?.value / 100, currencyCode!, locale))

[carlosviniciusananias]

@mcandeia makes a lot of sense... improvements completed

[mcandeia]

Do not merge yet, I’ll fix the namespace bug that makes the namespace be your fork repo

[mcandeia]

Suggested change

	* @description Default message displayed when I don't have free shipping
	* @description The *template* that will be used for mounting the free shipping message when its not reached

[mcandeia]

Please, can you add an example with @example as I mentioned previously? Thanks

[carlosviniciusananias]

Please, can you add an example with @example as I mentioned previously? Thanks

yes, done

[mcandeia]

Suggested change

	* @description Message displayed when I have free shipping
	* @description The message used when free shipping is reached

[mcandeia]

Why not replace {currency} as well?

[carlosviniciusananias]

Why not replace {currency} as well?

the formatPrice() function already delivers the formatted price respecting the currency... there is no need to substitute in currency.

formatPrice() ... R$ X,XX

[mcandeia]

Would you mind to remove it manually? Looks like you’re working in your own fork, which is fine, but there’s a bug that adds your repistory here so you need to remove it manually.

note that you should do it as a last task since your “deno task start” will add it again

[mcandeia]

Do not merge yet, I’ll fix the namespace bug that makes the namespace be your fork repo