AzUrlShortener icon indicating copy to clipboard operation
AzUrlShortener copied to clipboard

Solutions to Issues #31, #48 and Additional Code Refactoring

Open mreyeros opened this issue 4 years ago • 5 comments

  • Introduced Blazored.FluentValidation for handling Validation of ShortUrlEntity on Edits or Adds
  • Introduced IStorageTableHelper to inject StorageTableHelper via Dependency Injection for all Azure Functions removing code duplication
  • Modified ShortUrlEntity Clicks definition from int to Dictionary<string, int> to store click counts by country
  • Intriduced some helpers and extensions methods to reduce code duplication
  • Introduced EntityJsonPropertyConverter to handle serialization/deserialization of ShortUrlEntity Clicks property
  • Introduced UserIpLocationService to call the https://freegeoip.app/ service for obtaining user click data (i.e. country of origin)
  • Updated all azure functions to inject IStorageTableHelper to reduce code duplication and coupling to StorageTableHelper implementation

mreyeros avatar Oct 19 '20 10:10 mreyeros

Wow very nice work @mreyeros !!!

Let's plan some session to test all those super cool new features.

FBoucher avatar Dec 09 '20 16:12 FBoucher

:) You are welcome Frank, absolutely lets do it!

mreyeros avatar Dec 09 '20 20:12 mreyeros

Looking good

DavidTCarpenters avatar Jul 30 '21 14:07 DavidTCarpenters

Needs the lastest from branch vnext merging into and conflicts resolved

DavidTCarpenters avatar Jul 30 '21 14:07 DavidTCarpenters

FYI @DavidTCarpenters I started the v3 branch to use the Identity of SWA and really simplify the deployment.

FBoucher avatar Jul 30 '21 15:07 FBoucher