Skip to content

Feat(accounts): Add account_statements resource to v2 accounts#22

Merged
JoMarchant merged 3 commits intomainfrom
trnf-6630-actualizar-sdk-ruby
Apr 7, 2026
Merged

Feat(accounts): Add account_statements resource to v2 accounts#22
JoMarchant merged 3 commits intomainfrom
trnf-6630-actualizar-sdk-ruby

Conversation

@JoMarchant
Copy link
Copy Markdown
Contributor

@JoMarchant JoMarchant commented Mar 31, 2026

Contexto

Parte del proyecto de Estados de Cuenta IFPE. Se agrega soporte en el SDK de Ruby para el nuevo recurso account_statements bajo cuentas v2.

Qué hay de nuevo?

  • Se agrega el recurso AccountStatement y su manager AccountStatementsManager como recurso anidado bajo la clase Account de v2, permitiendo listar estados de cuenta via account.account_statements.list.
  • [BONUS] Se agrega el recurso Movement y su manager MovementsManager como recurso anidado bajo la clase Account de v2. Este endpoint no estaba 😨

Tests

  • Unitarios

Add AccountStatement resource and AccountStatementsManager to support
the new GET /v2/accounts/{account_id}/account_statements endpoint.
@JoMarchant JoMarchant self-assigned this Mar 31, 2026
@JoMarchant JoMarchant force-pushed the trnf-6630-actualizar-sdk-ruby branch 2 times, most recently from 0cd51bd to 7d27c59 Compare March 31, 2026 19:25
Add specs for AccountStatement resource, AccountStatementsManager,
and account_statements lazy accessor on Account.
@JoMarchant JoMarchant force-pushed the trnf-6630-actualizar-sdk-ruby branch from 7d27c59 to f963451 Compare March 31, 2026 19:28
Add Movement resource and MovementsManager to support
the GET /v2/accounts/{account_id}/movements endpoint.
@JoMarchant JoMarchant requested review from FelipeCampbell and pibahamondesw and removed request for pibahamondesw March 31, 2026 20:04
@JoMarchant JoMarchant marked this pull request as ready for review March 31, 2026 23:57
@JoMarchant
Copy link
Copy Markdown
Contributor Author

@FelipeCampbell del futuro, probé todos los SDKs en local e2e con estados de cuenta fake y está todo funcionando :pepe-gucci:

@JoMarchant
Copy link
Copy Markdown
Contributor Author

@FelipeCampbell cuando puedas plox

Copy link
Copy Markdown

@FelipeCampbell FelipeCampbell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@JoMarchant JoMarchant merged commit e256cbc into main Apr 7, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants