You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I v6, I propose we drop the GridEvents enum in favor of a simple type generated from the keys of GridEventLookup (the actual GridEventsStr.
That way, we will not have to define the pro-only and premium-only events inside the gridEvents.ts file which is located in the community package.
Just like for pre-processor, there is no reference to pro-only processors inside the community package because they are based on the lookup which is augmented in the pro and premium packages.
I v6, I propose we drop the
GridEvents
enum in favor of a simple type generated from the keys ofGridEventLookup
(the actualGridEventsStr
.That way, we will not have to define the pro-only and premium-only events inside the
gridEvents.ts
file which is located in the community package.Just like for pre-processor, there is no reference to pro-only processors inside the community package because they are based on the lookup which is augmented in the pro and premium packages.
Goals
Work to do
GridEvents
in our codebase and docs (replace with the string which is already supported)GridEvents
as deprecated (I don't see how we could do a runtime warning)GridEvents
toGridEventLookup
and updatebuildGridEventsDocumentation
GridEvents
The text was updated successfully, but these errors were encountered: