V1pi / iugu-node-sdk

BSD 3-Clause "New" or "Revised" License
14 stars 7 forks source link

Map para params #5

Closed incompletude closed 2 years ago

incompletude commented 4 years ago

Cara, esse design é meio estranho saca? Como é o oneliner? Não sei se tem.

    const params: Map<string, string> = new Map()
    params.set("customer_id", idIuguClient)

Não seria melhor algo como { [key: string]: string }, até mesmo { [key: "prop1" | "prop2"]: string }?

O código final fica:

const params = {
  "prop1": "valor"
}

Posso mandar 1 pr?

V1pi commented 4 years ago

Pode mandar um pull request. Obrigado pela contribuição.

V1pi commented 2 years ago

Estou trabalhando nessa funcionalidade na PR https://github.com/V1pi/iugu-node-sdk/pull/30

V1pi commented 2 years ago

Implementado https://github.com/V1pi/iugu-node-sdk/pull/30