Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request: Create Exchange Rate Updater Project and Add Czech National Bank Connector
Overview
My main aim was to build a project that's easy to understand, use, and maintain. This led to the creation of two projects:
ExchangeRateUpdater
andExchangeRateUpdater.UnitTests
.Changes Made
General
Program.cs
.appsettings.json
and made it easier to access with a dedicated class.Architecture
Followed the Clean Architecture approach:
Testing
Added the
ExchangeRateUpdater.UnitTests
project to check some logic of the application.How to Review
ExchangeRateUpdater
folder: usedotnet run
.ExchangeRateUpdater
folder: usedotnet test
.Development Environment
Created on a Windows 10 computer using Visual Studio 2022 Preview 17.5.0 Preview and .NET 7.
Additional Notes
I aimed to make this solution scalable with the Factory Pattern, but I also tried to keep it as simple as possible (that's why I used folders instead of separate projects, for instance).