fio-api-php
fio-api-php copied to clipboard
mhujer
Možnost zadání transakcí k provedení.
FIO API umožňuje zadat transakci či balík transakcí k provedení (resp. k pozdějším autorizování celého baliku z internetbankingu smskou). Je v plánu tuto funkcionalitu implementovat do fio-api-php ?
Když jsem knihovnu vytvářel, tak jsem potřeboval jen stahovat a párovat došlé platby. Odchozí jsem zatím nepotřeboval, takže se je implementovat nechystám. Ale samozřejmě PR uvítám :-)
Ahoj @ge0rgecz, pokud potřebuješ provádět aktivní transakce, tak můžeš použít náš fork této knihovny - https://github.com/perlur/php-fio-api; přes composer nás najdeš pod perlur/php-fio-api a verze je dev-master, ale budeme releasovat 3.0.0 až dopíšeme mezinárodní platby, zatím máme jen domestic a SEPA.
Dobry den @mark-stopka, skvela prace! Proc jste neudelali PR do tohoto repa?
Dobry den @mhujer , mam pripravenu branch s podporou uploadov, zalozenu na praci @Petr Kramar
od @mark-stopka. Slo o aplikaci jejich zmen na nejnovejsi verzi teto knihovny, plus nejake opravy, aktualizace readme, atp. Jste otevren mergu? Muzu nato udelat PR? Je to k nahlidnuti tady https://github.com/FELDSAM-INC/fio-api-php/tree/feldsam-uploads
@feldsam On už tu jeden takový hezky udělaný PR https://github.com/mhujer/fio-api-php/pull/25 od @jusephe, plánovali jsme to nasadit v Donio, ale tam už mezitím nejsem, takže nevím, jestli to je otestované v produkci. @jusephe, už to máte nasazené nebo ještě ne? :-)
Já teď knihovnu aktuálně na žádném projektu nepoužívám, takže jsem do ní neinvestoval moc času a zasloužila by nejdřív nasadit PHPStan a pak teprve dělat větší změny. Takže pokud by se vám do toho chtělo - udělat jeden PR na PHPStan a nad tím pak PR na ty změny pro zakládání plateb, tak by to bylo super. (Případně to nějak nakombinovat s tím, jak je to udělané v #25)
Sel bych cestou pridani phpstan a pak tych zmen co jsem jiz udelal. Mam to uz nasadene v produkci na zadavani prikazu k inkasu. Jeste tam chci dodelat podporu pro sepa direct debit - tam je podle dokumentace kompletne jine xml
@mhujer Nasazené to ještě nemáme, a myslím, že ještě dlouho nebudeme mít :)
@mhujer Zdravim, udelal jsem PR #32 na ten phpstan. Muze to tak byt? Mrknete nato? Dekuji
@feldsam Super, mergnuto, díky! Rovnou jsem udělal followup https://github.com/mhujer/fio-api-php/pull/33 a nastavil ho ještě striktněji.