search

mhujer / fio-api-php

PHP implementation of Fio Bank API
MIT License
32 stars 31 forks source link

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

Open ge0rgecz opened 8 years ago

ge0rgecz commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

@marek-stopka :+1:

feldsam commented 5 months ago

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

feldsam commented 4 months ago

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 commented 4 months ago

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

feldsam commented 4 months ago

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 commented 4 months ago

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

feldsam commented 4 months ago

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

mhujer commented 4 months ago

@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.