Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Types International transfers #205

Closed
wants to merge 6 commits into from
Closed

Types International transfers #205

wants to merge 6 commits into from

Conversation

rogelioLpz
Copy link
Member

@rogelioLpz rogelioLpz commented Feb 21, 2022

@rogelioLpz rogelioLpz added the enhancement New feature or request label Feb 21, 2022
@rogelioLpz rogelioLpz self-assigned this Feb 21, 2022
@codecov
Copy link

codecov bot commented Feb 21, 2022

Codecov Report

Merging #205 (1b226cc) into main (a7381f7) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #205   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           13        13           
  Lines          992      1018   +26     
=========================================
+ Hits           992      1018   +26     
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
cuenca_validations/types/__init__.py 100.00% <ø> (ø)
cuenca_validations/types/enums.py 100.00% <100.00%> (ø)
cuenca_validations/types/queries.py 100.00% <100.00%> (ø)
cuenca_validations/types/requests.py 100.00% <100.00%> (ø)
cuenca_validations/version.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a7381f7...1b226cc. Read the comment docs.

@rogelioLpz rogelioLpz changed the title transfers Types transfers Feb 23, 2022
@rogelioLpz rogelioLpz changed the title Types transfers Types International transfers Feb 23, 2022
Copy link
Member

@alexviquez alexviquez left a comment

Choose a reason for hiding this comment

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

unas preguntas

Comment on lines +409 to +416
bank_number: str
account_number: str
account_country: Country
account_name: str
received_amount: int
received_currency: Currency
sent_amount: int
sent_currency: Currency
Copy link
Member

Choose a reason for hiding this comment

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

el campo de arch route no va a venir aquí? o no es necesario en este modelo

Comment on lines +410 to +412
account_number: str
account_country: Country
account_name: str
Copy link
Member

Choose a reason for hiding this comment

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

objeto account con number, country, name

Copy link
Member

Choose a reason for hiding this comment

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

bank_number

Comment on lines +413 to +415
received_amount: int
received_currency: Currency
sent_amount: int
Copy link
Member

Choose a reason for hiding this comment

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

porque necesitas esto?

@rogelioLpz rogelioLpz closed this Jun 27, 2022
@rogelioLpz rogelioLpz deleted the in-transfers branch September 14, 2022 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Types International Transfers
4 participants