Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Formát dat v API rozšíření o zpracování a vypořádání transakce #639

Closed
pvlst opened this issue Apr 28, 2022 · 2 comments
Closed
Assignees
Labels
Milestone

Comments

@pvlst
Copy link

pvlst commented Apr 28, 2022

Rozšíření „Detailní datum a čas zpracování a vypořádání transakce“ obsahuje 4 různé formáty data a času. Z jakého důvodu není dodržen jednotně např. ISO 8601 formát, tzn. YYYY-MM-DD pro datum a YYYY-MM-DDThh:mm:ss+-hh:mm pro datum včetně časové složky?

Kvůli tomuto musíme implementovat 3 různé manuální konvertory těchto dat, neboť vyjma položky createdDate jsou ostatní formáty nestandardní. Plus to také naráží na problém popsaný v jiném issue s neuváděním časových zón: #644

@dmarek
Copy link
Contributor

dmarek commented Sep 7, 2022

Dané rozšíření vzniklo poměrně dávno a nebylo designováno s ohledem na ISO 8601. Změny v tomto rozšíření neplánujeme.

@dmarek dmarek closed this as completed Sep 7, 2022
@janbrasna janbrasna transferred this issue from another repository Sep 16, 2022
@janbrasna janbrasna added this to the v2+ milestone Sep 16, 2022
@janbrasna janbrasna self-assigned this Sep 16, 2022
@janbrasna janbrasna closed this as not planned Won't fix, can't repro, duplicate, stale Sep 16, 2022
@janbrasna
Copy link
Contributor

@pvlst To samozrejme irituje vsechny, nicmene jednotlive formaty se za posledni dekadu postupne posbiraly "cestou" z jinych systemu, kombinaci z jinych rozsireni atp. ze kterych toto verejne vznikalo, to je ale v realite amalgam lecjakych privatnich potreb obchodniku za ta leta…

Vsechny nove hodnoty se pridavaji jako ISO 8601… nicmene historicke formaty zustavaji pro zpetnou kompatibilitu napric eAPI v1.x; urcite jakmile bude na poradu dne backlog pro breaking change na API v2+ tak podobne roztrousene formaty se budou sjednocovat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants