Skip to content

Isaac Rossell I Ripoll Take home #774

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
Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

Isaac Rossell I Ripoll Take home #774

wants to merge 27 commits into from

Conversation

ZakRipoll
Copy link

Old school, design patter oriented solution:

  1. Chain of Responsibility, to check every step of the way is the task can be handled
  2. Decorator, to keep adding functionality each of step oriented to just 1 outcome
  3. Singleton, to keep a single instance

ZakRipoll and others added 27 commits October 3, 2025 19:00
@ZakRipoll
Call to CNB
ccc6d73
@ZakRipoll
CNB result object
e5b6722
@ZakRipoll
Read line by line
8f9bf2d
@ZakRipoll
From List to Dictionary
715640a
@ZakRipoll
Use the dictionary
baa88a7
@ZakRipoll
rate from double to decimal
3b35fc4
@ZakRipoll
rate from double to decimal
e21469a
@ZakRipoll
Merge branch 'solution' of https://github.com/ZakRipoll/take-home int…
8ca946a 
…o solution
@ZakRipoll
Ignore .vs
f6f0264
@ZakRipoll
gitignore
ae37761
@ZakRipoll
Decorator pattern
048cb9c
@ZakRipoll
Singleton db
ebc656e
@ZakRipoll
Apply Singleton
eff5534
@ZakRipoll
Add the . as a decimal
2f202fe
@ZakRipoll
Add DataClean in between APICall and LoadData
071a2f6 
Add regex to ensure the data is DD.MM.YYYY in APICall, to call a day different than today
CleanData uses regex to only accept the proper data
Load data only has to insert the filtered data
@ZakRipoll
Add attributes
fb4f41e
@ZakRipoll
Chain of Responsibility
08dd5b1
@ZakRipoll
Add IsEmpty
4c2dd6d
@ZakRipoll
Add the Chain of Responsibility behavior
54ed278
@ZakRipoll
Moved the url to appsettings.json
d040504
@ZakRipoll
Add basic testing for Decorator
9977225
@ZakRipoll
Test LoadData
a8f4fc1
@ZakRipoll
Test DataClean
5d14f93
@ZakRipoll
Test APICall
aca3511
@ZakRipoll
Test Chain of Responsibility
0726a7f
Changes
740eeef
Changes
6f37420
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

@ZakRipoll