xseman / bysquare

Encode & Decode "PAY by square" QR string.
https://xseman.github.io/bysquare/
Apache License 2.0
31 stars 6 forks source link

Chybaju dalsie polia #31

Closed fafayqa closed 1 month ago

fafayqa commented 5 months ago

Bolo by fajn, keby tento kod obsahoval VSETKY POLIA, ktore je mozne vyplnit, nie len tie 3 ci 4 (Sprava pre prijimatela, Nazov prijimatela, Datum vykonania/splatnosti, Specificky symbol, Konstantny symbol). Alebo povedane inak: bolo by dobre vediet, ake su nazvy pre chybajuce polia v tomto kode:

payments: [
    {
        type: PaymentOptions.PaymentOrder,
        amount: this._ammount,
        bankAccounts: [{ iban: this._iban }],
        currencyCode: CurrencyCode.EUR,
        variableSymbol: this._variable
    }
]

Minimalne pole "Sprava pre prijimatela" je totalna nutnost, bez neho nemozem sparovat platbu s e-mailom (v sprave mi totiz klient uvadza svoj e-mail a tak to viem sparovat, dohladat a kontaktovat ho).

xseman commented 5 months ago

Ak správne chápem, používaš JavaScript bez typov a nedostávaš hinty. Momentálne robím s VSCode, ktorý vie v JS typy hintovať, ak ich knižnica obsahuje. Ak nemáš hinty o typoch, pozri sa na typ SimplePayment, kde sú definované základné dáta pre platby.

https://github.com/xseman/bysquare/blob/master/src/types.ts#L177

github-actions[bot] commented 2 months ago

This issue is stale because it has been open for 90 days with no activity.

github-actions[bot] commented 1 month ago

This issue was closed because it has been inactive for 30 days since being marked as stale.

xseman commented 1 month ago

@lukasbicus @dderevjanik Nejaký nápad ako zlepšiť dokumentáciu? Nerád by som dával obrovské examples do hlavného README

lukasbicus commented 1 month ago

@xseman - suhlasím s tým, že README má byť stručné. Zároveň si tiež myslím, že pre @fafayqa a iných užívateľov by bolo naozaj prínosné vedieť, či tento package vie uspokojiť ich potreby ešte predtým, než si ho nainštalujú a vidia nápovedu v IDE.

Riešením by mohlo byt vygenerovať z JSdoc dokumentáciu a odkázať naňu z README.md. Prvé video, ktoré mi podhodil youtube s návodom je napr. toto: https://www.youtube.com/watch?v=FrUQnR4GTpQ

xseman commented 1 month ago

JSdoc vyzerá fajn, dosť mi to pripomína TypeDoc. Nad generovaním dokumentácie som taktiež rozmýšlal, no typ človeka ktorý otvorí takúto dokumentáciu už skor nainštaluje package, alebo pozrie zdrojáky interface kde to uvidí podobne ako v takejto dokumentácí.

@lukasbicus čo povieš na to, že upravíme https://github.com/xseman/bysquare?tab=readme-ov-file#adavanced-usage dali by sme ho ako <details> element so všetkými vyplnenými fields, tým padom bude schovaný a nezaberie to toľko miesta v README.

lukasbicus commented 1 month ago

Nepodceňoval by som potrebu užívateľov mať plnohodnotnú dokumentáciu. Samozrejme použiť details je tiež cesta.