Damyan Petev

@kdinev Not quite, the current change for #1067 only creates multiple error messages (through the validator, notifier itself being completely unaware). As I see it, what this describes would require...

@StefanIvanov For now we've scheduled adding a map template to components. My concern is still with the structure for the "Charts" - currently both our sites and the CLI structure...

**TL;DR:** We should probably postpone this and #194 for future updates in the Ignite UI for JS packages. I've made a [test project repo](https://github.com/damyanpetev/cli-react-webpack-test/commits/master) to test out loading modules. Besides...

Since `9` is one of the [standard mask characters](https://www.infragistics.com/products/ignite-ui-angular/angular/components/mask.html#supported-built-in-mask-rules) that specifies a digit input, that's sort of the expected behavior. As the Mask directive currently doesn't support escaping mask characters,...

@kdinev I agree, that's why the issue is still open :) Whether the static part is part of the value is based on use, but fact is atm there no...

@wnvko The argument was explicitly renamed from `key` to `primaryKey` because it was not the same type - as in it'll never switch to the row object to use as...

Oh, @skrustev forgot - a changelog would be nice too :)

LGTM, though this missed the window for 17.2 and will likely land in 18.0 so the changelog will need an update. No rush though. Also, @simeonoff If you can quickly...

Small correction on the actual result: It's not that the Grid doesn't span to 100% - it does. Actually, since the `display: grid` handles the internal structure the actual grid...

When you say the `AngularTypeSriptFileUpdate` "extends", by the current API exposed this is more geared toward composition, rather than inheritance btw. None of the AST-related API are especially friendly to...