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

Git exercise - Aniruddha Sahasrabuddhe #81

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Git exercise - Aniruddha Sahasrabuddhe #81

wants to merge 2 commits into from

Conversation

anirus-engg
Copy link

Includes the following changes:

  • Refactored the code to fix bugs and improve logic
  • New request: A customer can transfer between their accounts
  • New request: Change Maxi-Savings accounts to have an interest rate of 5% assuming no withdrawals in the past 10 days otherwise 0.1%
  • New request: Interest rates should accrue daily (incl. weekends), rates above are per-annum
  • New feature: Interest is deposited to the account total and is displayed in the account statement.

@anirus-engg
fixed bugs in the current logic and added requirement to transfer bet…
664491f 
…ween accounts
@anirus-engg
added enhancemens for:
eb8a13c 
Change Maxi-Savings accounts to have an interest rate of 5% assuming no withdrawals in the past 10 days otherwise 0.1%
Interest rates should accrue daily (incl. weekends), rates above are per-annum
Interest is added to the account total when interest paid is calculated.  Statement then includes the deposited interest.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@anirus-engg