Skip to content

[EXP-3223] Add support for cash payment method#91

Merged
Fenho merged 3 commits into
masterfrom
feat/create-cash
Jul 10, 2025
Merged

[EXP-3223] Add support for cash payment method#91
Fenho merged 3 commits into
masterfrom
feat/create-cash

Conversation

@Fenho

@Fenho Fenho commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

Al final solo arreglo los tests para que hablen de payment_type en vez de payment_method

@Fenho Fenho force-pushed the feat/create-cash branch from 683de7d to 58a9eab Compare July 9, 2025 21:09
@Fenho Fenho marked this pull request as ready for review July 9, 2025 21:28
@Fenho Fenho requested review from daleal and lezorich July 9, 2025 21:51

@lezorich lezorich left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Fenho lo voy a aprobar, pero el test de cash no es necesario. Solo le cambiaría lo del payment_method a payment_type.

El test no es necesario porque envdd no testea nada que no esté testeado en otros lados

Comment thread tests/test_integration.py Outdated
@Fenho Fenho merged commit 01668d4 into master Jul 10, 2025
5 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