Update skyAppLink to use optional excludeDefaults to getAll.

[Blackbaud-BobbyEarl]

Addressing blackbaud/skyux2#1650

[codecov-io]

Codecov Report

Merging #402 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #402      +/-   ##
=========================================
+ Coverage    99.3%   99.3%   +<.01%     
=========================================
  Files          73      73              
  Lines        1867    1873       +6     
  Branches      293     294       +1     
=========================================
+ Hits         1854    1860       +6     
  Misses         13      13

Flag	Coverage Δ
#builder	100% <ø> (ø)	⬆️
#runtime	95.42% <100%> (+0.09%)	⬆️
#srcapp	100% <ø> (ø)	⬆️

Impacted Files	Coverage Δ
runtime/directives/sky-app-link.directive.ts	100% <100%> (ø)	⬆️
runtime/params.ts	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6f23cdb...85a3a0f. Read the comment docs.

[Blackbaud-BobbyEarl]

Let me know if this implementation is more inline what what you were thinking @Blackbaud-PaulCrowder.

[Blackbaud-PaulCrowder]

My suggestion here would be to write a simple loop to build up the object. It'll cut down on both the amount of code written as well as the number of loops executed (1 vs. 2) and be more legible IMO.

const filteredParams: { [key: string]: string} = {};

this.getAllKeys().forEach(key => {
  if (!excludeDefaults || this.params[key] !== this.defaultParamValues[key]) {
    filteredParams[key] = this.params[key];
  }
});

return filteredParams;