Updating v34

.github/workflows/change-log.yml

@@ -0,0 +1,62 @@
+# yaml
+name: Check changelog entry
+
+on:
+  pull_request:
+    types: [opened, synchronize, reopened, labeled, unlabeled]
+
+permissions:
+  contents: read
+jobs:
+  check-changelog:
+    # Allow intentional bypass by adding the `skip-changelog` label to the PR.
+    if: ${{ !contains(github.event.pull_request.labels.*.name, 'skip-changelog') }}
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Determine changed files and require changelog update
+        env:
+          EVENT_NAME: ${{ github.event_name }}
+          PR_BASE_REF: ${{ github.event.pull_request.base.ref || '' }}
+          BEFORE: ${{ github.event.before }}
+          SHA: ${{ github.sha }}
+        run: |
+          set -euo pipefail
+
+          echo "Event: $EVENT_NAME"
+
+          if [ "$EVENT_NAME" = "pull_request" ]; then
+            if [ -z "$PR_BASE_REF" ]; then
+              echo "Pull request base ref not provided."
+              exit 1
+            fi
+            echo "Fetching base ref origin/$PR_BASE_REF"
+            if ! git fetch origin "$PR_BASE_REF":refs/remotes/origin/"$PR_BASE_REF"; then
+              echo "::error::Failed to fetch pull request base ref origin/$PR_BASE_REF."
+              exit 1
+            fi
+            changed=$(git diff --name-only "origin/$PR_BASE_REF...HEAD")
+          else
+            if [ "$BEFORE" = "0000000000000000000000000000000000000000" ]; then
+              # new branch or forced push: list files in the commit
+              changed=$(git show --name-only --pretty="" "$SHA")
+            else
+              changed=$(git diff --name-only "$BEFORE...$SHA")
+            fi
+          fi
+
+          echo "Changed files:"
+          echo "$changed" | sed 's/^/ - /'
+
+          # match: CHANGELOG.md, NEWS.md, any file under changelog(s)/ or docs/changelog/
+          echo "$changed" | grep -E '(^CHANGELOG\.md$|^NEWS\.md$|(^|/)changelog(/|$)|(^|/)changelogs(/|$)|(^|/)docs/changelog(/|$))' >/dev/null 2>&1 || {
+            echo "::error::No changelog changes detected."
+            echo "Please add or update a changelog entry (e.g. update `CHANGELOG.md` or add a file under `changelog/`)."
+            exit 1
+          }
+
+          echo "Changelog change detected. OK."

.github/workflows/npm-publish.yml

@@ -0,0 +1,47 @@
+name: Publish to NPM
+
+on:
+  release:
+    types: [published]
+  workflow_dispatch:
+
+jobs:
+  build:
+    uses: ./.github/workflows/build.yml
+    with:
+      python-version: '3.10'
+      node-version: 'v18.16.0'
+
+  npm-publish:
+    name: Upload release to NPM
+    runs-on: ubuntu-latest
+    environment:
+      name: npm
+    permissions:
+      id-token: write
+      contents: read
+    needs: build
+    steps:
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '20.x'
+          registry-url: 'https://registry.npmjs.org'
+      - name: Upgrade npm for trusted publishing
+        # Trusted publishing (OIDC) requires npm >= 11.5.1, newer than the
+        # version bundled with Node 20.
+        run: npm install -g npm@latest
+        shell: bash
+      - name: Download npm package
+        uses: actions/download-artifact@v4
+        with:
+          name: npm-package
+          path: npm-dist/
+      - name: Publish to npm
+        # Authenticates via GitHub OIDC using the trusted publisher configured
+        # on npmjs.com — no NPM_TOKEN needed. Provenance is generated automatically.
+        run: |
+          pushd npm-dist
+          FILE=$(echo *.tgz)
+          npm publish "$FILE" --access public
+          popd
+        shell: bash

.github/workflows/publish.yml → .github/workflows/pypi-publish.yml

@@ -1,4 +1,4 @@
-name: Publish to PyPI & NPM
+name: Publish to PyPI
 
 on:
   release:
@@ -30,22 +30,3 @@ jobs:
           path: dist/
       - name: Publish package distributions to PyPI
         uses: pypa/gh-action-pypi-publish@release/v1
-
-      - name: Download npm package
-        uses: actions/download-artifact@v4
-        with:
-          name: npm-package
-          path: npm-dist/
-      - uses: actions/setup-node@v4
-        with:
-          node-version: '20.x'
-          registry-url: 'https://registry.npmjs.org'
-      - name: Publish to npm
-        run: |
-          pushd npm-dist
-          FILE=$(echo *.tgz)
-          npm publish "$FILE" --provenance --access public
-          popd
-        env:
-          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
-        shell: bash

.github/workflows/python-test.yml

@@ -8,10 +8,14 @@ on:
     branches: [main]
     tags:
       - v*
+    paths:
+      - src/**
   pull_request:
     branches:
       - main
       - v*
+    paths:
+      - src/**
 
 jobs:
   test:

CHANGELOG.md

@@ -5,11 +5,16 @@ This project adheres to [Semantic Versioning](https://semver.org/).
 Links "DE#nnn" prior to version 2.0 point to the Dash Enterprise closed-source Dash AG Grid repo
 
 ## [unreleased]
-
-### Added 
+### Added
 - [#436](https://github.com/plotly/dash-ag-grid/pull/436) Enabled Filter Handlers to simplify custom filter components by splitting the filter logic out from the UI component.
 - [#440](https://github.com/plotly/dash-ag-grid/pull/440)
   - added `columnTypes` to prop categories for parsing functions
+- [#453](https://github.com/plotly/dash-ag-grid/pull/453) Test for changelog entry
+- [#448](https://github.com/plotly/dash-ag-grid/pull/448) Added support for AG-Charts (split out in v33 of AG Grid).  Integrated Charts require enableEnterpriseModules=True, dashChartMode="community" or "enterprise", and dashGridOptions={"enableCharts": True}.
+- [#455](https://github.com/plotly/dash-ag-grid/pull/455) Added support for dynamic `detailCellRendererParams` in Master/Detail, including dynamic detail-grid column definitions.
+- [#468](https://github.com/plotly/dash-ag-grid/issues/468) Support passing a function to the grid option `processUnpinnedColumns`.
+- [#472](https://github.com/plotly/dash-ag-grid/pull/472) Added support for callback function for the groupAggFiltering options
+
 
 ### Changed
 - [#440](https://github.com/plotly/dash-ag-grid/pull/440) 
@@ -18,8 +23,22 @@ Links "DE#nnn" prior to version 2.0 point to the Dash Enterprise closed-source D
     - `_self` will auto apply as a `linkTarget`.
     - `_blank` will auto apply `rel='noreferrer noopener 'nofollow'` for security reasons, but will open in a new tab.
   - version bump to v`35.2.0` for the grid
+- [#452](https://github.com/plotly/dash-ag-grid/pull/452) 
+  - Added test for `OBJ_MAYBE_FUNCTION_OR_MAP_MAYBE_FUNCTIONS` to test that the value is an object before parsing, 
+  - this allows for reused keys to not comply with being an object
+
+
+### Fixed
+- [#454](https://github.com/plotly/dash-ag-grid/pull/454) fixes issue where a rowCount of 0 would cause the grid not to display new data
+- [#459](https://github.com/plotly/dash-ag-grid/pull/459) removes `getRowsRequest` / `getRowsResponse` warnings by excluding these Dash callback props from AG Grid options.
+- [#460](https://github.com/plotly/dash-ag-grid/pull/460) fix grid rendering when `getRowId` is explicitly set to `None`
+- [#466](https://github.com/plotly/dash-ag-grid/issues/466) guard against a null `gridApi` in `onSortChanged`, which threw an uncaught `TypeError` when a grid initialized with a sort applied via `initialState.sort.sortModel`
 
 ## [34.0.0rc0] - 2026-01-21
+### Changed
+- bumping to v`34.0.0` for the grid
+
+## [33.3.3] - 2025-12-19
 ### Fixed
 - [#408](https://github.com/plotly/dash-ag-grid/pull/408) fixed issue where the `columnState` would conflict with `columnDefs` updates
     - fixes [#416] (https://github.com/plotly/dash-ag-grid/issues/416)

docs/examples/enterprise/master_detail.py

@@ -26,6 +26,11 @@
     {"headerName": "Pop. (Metro area)", "field": "population_metro"},
 ]
 
+detailColumnDefsSimple = [
+    {"headerName": "City", "field": "city"},
+    {"headerName": "Pop. (City proper)", "field": "population_city"},
+]
+
 rowData = [
     {
         "country": "China",
@@ -164,6 +169,31 @@
             ),
             body=True,
         ),
+        html.Hr(),
+        dbc.Card(
+            dcc.Markdown(
+                "Use a JavaScript function in `detailCellRendererParams` to dynamically define detail columns per expanded row."
+            ),
+            body=True,
+        ),
+        dbc.Card(
+            dag.AgGrid(
+                id="master-detail-table-dynamic-columns",
+                columnDefs=masterColumnDefs,
+                rowData=rowData,
+                columnSize="sizeToFit",
+                enableEnterpriseModules=True,
+                masterDetail=True,
+                detailCellRendererParams={
+                    "function": """params.data.region === "Asia"
+                        ? {detailGridOptions: {columnDefs: %s}, detailColName: "cities", suppressCallback: true}
+                        : {detailGridOptions: {columnDefs: %s}, detailColName: "cities", suppressCallback: true}"""
+                    % (detailColumnDefsSimple, detailColumnDefs)
+                },
+                dashGridOptions={"detailRowAutoHeight": True},
+            ),
+            body=True,
+        ),
     ]
 )