Možnost zadání transakcí k provedení.

[ge0rgecz]

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 ?

[mhujer]

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 :-)

[mark-stopka]

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.

[mhujer]

@marek-stopka 👍

[feldsam]

Dobry den @mark-stopka, skvela prace! Proc jste neudelali PR do tohoto repa?

[feldsam]

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

[mhujer]

@feldsam On už tu jeden takový hezky udělaný PR #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)

[feldsam]

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

[jusephe]

@mhujer Nasazené to ještě nemáme, a myslím, že ještě dlouho nebudeme mít :)

[feldsam]

@mhujer Zdravim, udelal jsem PR #32 na ten phpstan. Muze to tak byt? Mrknete nato? Dekuji

[mhujer]

@feldsam Super, mergnuto, díky! Rovnou jsem udělal followup #33 a nastavil ho ještě striktněji.