Admin Generator (Future): Allow adding custom grid actions

[jamesricky]

---

PR Checklist

• Verify if the change requires a changeset. See CONTRIBUTING.md
• Link to the respective task if one exists: SVK-211
• Provide screenshots/screencasts if the change contains visual changes

Custom.Grid.Actions.-.View.Product.Details.mov

[nsams]

oh, and have you considered https://mui.com/x/react-data-grid/column-definition/#special-properties type: "actions" insted of our actions component?

[jamesricky]

oh, and have you considered https://mui.com/x/react-data-grid/column-definition/#special-properties type: "actions" insted of our actions component?

No, good point.

It seems like it could make sense to use this internally, but then we'd need to allow an array of component-imports as getActions() expects an array of individual action-components.
The current approach (in this PR) only needs a single component-import, as that can be a component with multiple actions.
A setting for the width would also still be needed.

[nsams]

but then we'd need to allow an array of component-imports as getActions() expects an array of individual action-components.

Yes, why not, it's always a good idea to stay close to MUI api

[jamesricky]

but then we'd need to allow an array of component-imports as getActions() expects an array of individual action-components.

Yes, why not, it's always a good idea to stay close to MUI api

If we would simply import and pass in a react component, like the MUI api expects I would agree.
In the generator however we need configuration for every component to import.
Since the configuration is just strings (name, import), the IDE doesn't know to check if the file or the component exists.
I think it would be good to avoid this kind of error-prone/non-autocompleted configuration as much as possible, that's why I'd prefer a single import with one component that has multiple IconButtons.

[jamesricky]

Setting this to "Draft" for now, until #2044 is approved.

The base branch was changed.

[johnnyomair]

LGTM. Please wait for @nsams review though.