search

Capgemini / xrm-datamigration-xrmtoolbox

Export and import data for Microsoft Dataverse as an XrmToolBox plugin. Supports JSON and CSV.
MIT License
8 stars 4 forks source link

refactor: remove `INotifier` #75

Closed danroche10 closed 2 years ago

danroche10 commented 2 years ago

A couple comments but generally pretty good! I like adding it to the helper class - keeps it reusable!

Comments all make sense! I've asked for clarification on one of them

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

100.0% 100.0% Coverage
2.9% 2.9% Duplication

danroche10 commented 2 years ago

A couple comments but generally pretty good! I like adding it to the helper class - keeps it reusable!

@tdashworth - implemented remaining change