[BUG] useModal pass metadata with show()

[aress31]

Describe the bug

Can a feature get implemented to pass metadata within useModalForm, this way id, action and any other relevant parameter can be passed down to the dialog component.

Steps To Reproduce

N/A

Expected behavior

show({foo, bar})

Packages

N/A

Additional Context

No response

[alicanerdurmaz]

Hello @aress31,

Can you please give me more details about your use case? You labeled it a bug. What issue or error did you encounter?

[aress31]

@alicanerdurmaz my bad should have been a feat, see the following use case:

const MySuperView = () => {
  // --- SNIP ---

  const [refineCoreProps, setRefineCoreProps] = useState();
  const modal = useModal();

  return (
    <Box>
      <DataGridComponent
        // --- SNIP ---
        slotProps={{
          toolbar: {
            modal: (action, id) => {
              setRefineCoreProps({ action, id });
              modal.show();
            },
          },
        }}
      />
      {modal.visible && <ModalForm modalProps={{ modal, refineCoreProps }} />}
    </Box>
  );
};

const DataGridToolbar = ({ modal }) => {
  /// --- SNIP ---

  const renderActions = () => (
    <>
      {/* --- SNIP --- */}
      <Button
        // --- SNIP ---
        onClick={() => modal("show", selectedRows.keys().next().value)}
      >
        Show
      </Button>
      <Button
        // --- SNIP ---
        onClick={() => modal("edit", selectedRows.keys().next().value)}
      >
        Edit
      </Button>
      <Button
        // --- SNIP ---
        onClick={() => modal("create")}
      >
        Create
      </Button>
    </>
  );

  return (
    <GridToolbarContainer>
      {/* --- SNIP --- */}
      {renderActions()}
    </GridToolbarContainer>
  );
};

Here I only pass action and id but in an ideal world we would have a meta prop - to stay consistent with the naming scheme to pass down to the ModalForm any relevant information.