ngChange after refreshComponent

[oezkany]

Hi,

you guys recently checked in a fix that when the model of a multiSelect component has changed, so refreshComponent will be called. Here: #1243

BUT:
It would be very nice if the refreshComponent would call a ngChange method. I.e. as a configurable directive attribute where we cann pass our pointer of method, which concrete method will be called.

Example:

<ui-select on-refesh="myNgChangeMethod">
  ...
</ui-select>

[rcrodrigues]

+1

[aaronroberson]

@oezkany I've merged in #1243 and it will land in the next release. However, we are not keen on adding a plethora of directive attributes to the project. I would love to see a PR that refactors the the directive away from having a million attributes and instead accepting a config object.

[oezkany]

@aaronroberson Thanks. Of course I will test it and will give you a feedback.

[aaronroberson]

By testing, you mean you'll write unit tests?