Sky Grid Calling Data Provider Multiple Times When Columns Change

[Blackbaud-TrevorBurch]

Expected behavior

When the columns are changed for a grid I would only expect one call at maximum to the data provider to get new info

Actual behavior

When the columns are changed for a grid with a data provider it is making multiple calls to the data provider which would potentially cause multiple async calls to get data for the grid.

Steps to reproduce

Setup a grid with initial columns
Change the columns in some manner to new columns
Observe multiple calls being made to the data provider (through logs or breakpoints)

Plunker

https://plnkr.co/edit/ATRZfehwjXOY9Ul0jxKe?p=preview

[Blackbaud-PatrickOFriel]

This is probably going to be pretty difficult to solve with the way getting and setting data in lists currently works (basically by watching changes to objects on observables). I think I'll try to solve this while working on this issue (#598) where I'm going to take a look at how the list interacts with data in general

[blackbaud-brandonstirnaman]

Have reproduced this in the ME Contrib version of this Grid as well. The gizmoduck team will take a look at this and fix in both controls. @blackbaud-joshgerdes @Blackbaud-LuisBello

[Blackbaud-LuisBello]

#982

the above PR provides a fix for this issue

[Blackbaud-TrevorBurch]

@Blackbaud-SteveBrush is this ready to be closed now?