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

Backend task dotnet thomas fisher #608

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

Conversation

Thomas-Fisher
Copy link

Implemented an ExchangeRateProvider for Czech National Bank while maintaining a structure that's open for additional providers.

  • Used the following API: https://api.cnb.cz/cnbapi/swagger-ui.html#/%2Ffxrates to retrieve exchange rates.
  • Created a small localhost API provider and client.
  • Integrated automatically generated Swagger documentation.
  • Created a factory to dynamically load providers.
  • Added NLog for both console and file logging.
  • Included validation for existing classes.
  • Created unit tests using xUnit.
  • Added README file to detail how to startup the projects

Implemented an ExchangeRateProvider for Czech National Bank while maintaining a structure that's open for additional providers.
- Used the following API: https://api.cnb.cz/cnbapi/swagger-ui.html#/%2Ffxrates to retrieve exchange rates.
- Created a small localhost API provider and client.
- Integrated automatically generated Swagger documentation.
- Created a factory to dynamically load providers.
- Added NLog for both console and file logging.
- Included validation for existing classes.
- Created unit tests using xUnit.
Added read me on how to start up the solution as the start project settings are stored in the local users settings and as such not committed to the GitHub repo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant