-
Notifications
You must be signed in to change notification settings - Fork 21
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
Money transfer validation #135
base: main
Are you sure you want to change the base?
Money transfer validation #135
Conversation
@stephane-segning |
What is the ticket you're working on? |
.../main/java/com/adorsys/gis/powerpay/powerpaybackend/errorhandling/TransactionRepository.java
Outdated
Show resolved
Hide resolved
...pay-backend/src/main/java/com/adorsys/gis/powerpay/powerpaybackend/services/Transaction.java
Outdated
Show resolved
Hide resolved
...nd/src/main/java/com/adorsys/gis/powerpay/powerpaybackend/services/MoneyTransferService.java
Show resolved
Hide resolved
f37562e
to
cb8fe29
Compare
…lement jpa in the MoneyTranferRepo
Hello @stephane-segning, i implemented all the changes you requested for this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see you're working well on this one. To help you continue in a good direction, let's discuss these points first:
...r-pay-backend/src/main/java/com/adorsys/gis/powerpay/powerpaybackend/domain/Transaction.java
Outdated
Show resolved
Hide resolved
...nd/src/main/java/com/adorsys/gis/powerpay/powerpaybackend/services/MoneyTransferService.java
Outdated
Show resolved
Hide resolved
...nd/src/main/java/com/adorsys/gis/powerpay/powerpaybackend/services/MoneyTransferService.java
Outdated
Show resolved
Hide resolved
...kend/src/main/java/com/adorsys/gis/powerpay/powerpaybackend/services/TransactionService.java
Show resolved
Hide resolved
Hello @stephane-segning fix: Refactor MoneyTransferService to improve validation and error ha… |
...kend/src/main/java/com/adorsys/gis/powerpay/powerpaybackend/services/TransactionService.java
Outdated
Show resolved
Hide resolved
...main/java/com/adorsys/gis/powerpay/powerpaybackend/repository/MoneyTransferValidateRepo.java
Show resolved
Hide resolved
...main/java/com/adorsys/gis/powerpay/powerpaybackend/repository/MoneyTransferValidateRepo.java
Outdated
Show resolved
Hide resolved
Hello @stephane-segning corrected the changes requested. |
I Implemented the send money validation