Skip to content

BDMS-775: Remove DataGrid Export and Type Contact Link Click Handler#276

Open
TylerAdamMartinez wants to merge 2 commits intostagingfrom
TAM-BDMS-775
Open

BDMS-775: Remove DataGrid Export and Type Contact Link Click Handler#276
TylerAdamMartinez wants to merge 2 commits intostagingfrom
TAM-BDMS-775

Conversation

@TylerAdamMartinez
Copy link
Copy Markdown
Contributor

@TylerAdamMartinez TylerAdamMartinez commented May 5, 2026

Why

This PR addresses the following problem/context:

  • Avoid showing the DataGrid export button when server-side pagination is enabled.
  • Make shared list toolbar actions configurable per page.
  • Improve TypeScript correctness for the contact link click handler.

How

Implementation summary - the following was changed/added/removed:

  • Added ListPageToolbarProps with optional hide flags for filter, columns, density, and export buttons.
  • Added toolbarOptions to ListPage so pages can optionally hide individual toolbar buttons.
  • Defaults all toolbar buttons to visible.
    • Hides the DataGrid export button by default when paginationMode is server.
  • Added an explicit React.MouseEvent<HTMLAnchorElement> type to the contact link onClick() handler.

Notes

Any special considerations, workarounds, or follow-up work to note?

  • Server-side/header export behavior is unchanged.
  • Pages can override the default export visibility through toolbarOptions.hideExport.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

Preview Deployment

Preview URL: https://preview-tam-bdms-775-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

@TylerAdamMartinez TylerAdamMartinez self-assigned this May 5, 2026
@TylerAdamMartinez TylerAdamMartinez added bug Something isn't working enhancement New feature or request labels May 5, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

Preview Deployment

Preview URL: https://preview-tam-bdms-775-auejgdbofq-uc.a.run.app

Note: This preview uses the staging API endpoints.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant