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.

Sign up for GitHub

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

Jump to bottom

Create core app logic #2

Closed
JanMate opened this issue Nov 19, 2021 · 1 comment · Fixed by #33
Closed

Create core app logic #2

JanMate opened this issue Nov 19, 2021 · 1 comment · Fixed by #33
Assignees
@Oleksandr6676

Comments

@JanMate
Copy link
Owner

JanMate commented Nov 19, 2021 •
edited
Loading

Create new class Transaction in logic/transaction folder with following methods:

  • init()
  • check_account_balance(account_id, amount)
  • transfer(sender_account, receiver_account, amount)
    • sender_account is Account instance
    • receiver_account is Account instance
Oleksandr6676 added a commit that referenced this issue Feb 9, 2022
@Oleksandr6676
feat: create core app logic
c7260cf 
it implements core app transaction logic

Closes #2
@Oleksandr6676 Oleksandr6676 mentioned this issue Feb 9, 2022
Merged
@JanMate
Copy link
Owner Author

JanMate commented Feb 11, 2022

@Oleksandr6676, sorry for misunderstanding. Now the assignment description is updated.

Oleksandr6676 added a commit that referenced this issue Feb 13, 2022
@Oleksandr6676
fix: application logic fixes
8513980 
adds account class, rework of the transaction class

Closes #2
Oleksandr6676 added a commit that referenced this issue Feb 15, 2022
@Oleksandr6676
fix: minor fixes and improvements
5c9f0ac 
adds TransactionException class, small fixes and code improvements

Closes #2
JanMate pushed a commit that referenced this issue Feb 15, 2022
@Oleksandr6676 @JanMate
feat: create core app logic
e71ddf7 
it implements core app transaction logic

Closes #2
JanMate pushed a commit that referenced this issue Feb 15, 2022
@Oleksandr6676 @JanMate
fix: application logic fixes
fe8d391 
adds account class, rework of the transaction class

Closes #2
JanMate pushed a commit that referenced this issue Feb 15, 2022
@Oleksandr6676 @JanMate
fix: minor fixes and improvements
7f91d72 
adds TransactionException class, small fixes and code improvements

Closes #2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Oleksandr6676 Oleksandr6676

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: create core app logic JanMate/happy_bank_core
2 participants
@JanMate @Oleksandr6676