Repository navigation
[BA-8916] Add transfer batches support to fintoc-ruby - #35
Conversation
|
|
||
| def to_s | ||
| amount_str = Money.from_cents(@transfer_amount, @currency).format | ||
| "📦 #{amount_str} (#{@id}) - #{@status}" |
There was a problem hiding this comment.
usamos emoticones en otra parte? no sea que rompa alguna integración
There was a problem hiding this comment.
sii en el sdk se usan jaja nose porque pero seguí el patrón nomas
| RSpec.shared_examples 'a client with transfer batches manager' do | ||
| let(:jws_private_key) do | ||
| key_string = "-----BEGIN PRIVATE KEY----- | ||
| MIIEvAIBADANBgkqhkiG9w0BAQEFAASCBKYwggSiAgEAAoIBAQDNLwwQr/uFToDH |
There was a problem hiding this comment.
porque necesitamos este JWT? contra que es validado?
There was a problem hiding this comment.
el POST de batches exige Fintoc-JWS-Signature igual que el de transfers, entonces el client lo firma con la llave que le pasas. En el spec es una llave de prueba pa que se pueda generar el header, nadie la valida por que VCR matchea solo method, uri y body. Es la misma que usan los examples de transfers y account verifications
rodrigoguerra-fintoc
left a comment
rodrigoguerra-fintoc
left a comment
There was a problem hiding this comment.
Solo esos dos comentarios que si son inválidos esta ok
Contexto
La API pública va a exponer transfer batches (
/v2/transfer_batches) y el SDK de Ruby todavía no los soporta.Que hay de nuevo?
client.v2.transfer_batchesconcreate,getylist, que devuelvenFintoc::V2::TransferBatch(conrefreshy predicados de status). Los montos sonamountysettled_amount, como en transfers, y traesucceeded_atypartially_succeeded_at.InvalidTransfersError,InvalidTransfersSizeError,InvalidDescriptionSizeError,InvalidSenderAccountsErroryOperationNotImplementedError. El SDK resuelve la clase por elcode, sin ellas tirabaNameError.Tests
Safety Checks
Consideraciones
/v2/transfer_batchesya está en main en fintoc-rails (BA-8905) y en sandbox (BA-8915).