Skip to content

feat(components/data-grid): create an easy mode for data grid#4063

Open
johnhwhite wants to merge 74 commits into13.x.xfrom
data-grid
Open

feat(components/data-grid): create an easy mode for data grid#4063
johnhwhite wants to merge 74 commits into13.x.xfrom
data-grid

Conversation

@johnhwhite
Copy link
Copy Markdown
Member

@johnhwhite johnhwhite commented Oct 31, 2025

@johnhwhite johnhwhite added the risk level (author): 1 No additional bugs expected from this change label Oct 31, 2025
@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Oct 31, 2025

View your CI Pipeline Execution ↗ for commit e8abe9f

Command Status Duration Result
nx build code-examples-playground --baseHref=ht... ✅ Succeeded 6m 21s View ↗
nx build playground --baseHref=https://blackbau... ✅ Succeeded 1m 39s View ↗
nx build integration --baseHref=https://blackba... ✅ Succeeded 11s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-09 17:47:32 UTC

@blackbaud-sky-build-user
Copy link
Copy Markdown
Collaborator

blackbaud-sky-build-user commented Oct 31, 2025

* origin/main:
  feat: add helper functions to get custom form errors and add SkyFormErrorHarness to docs (#4070)
  fix(components/ag-grid): maintain column types when using service grid options (#4078)
  fix: add missed links to global help docs (#4064)
  chore: release 13.7.2 (#4069)
  fix: update filter bar code examples to use writeable signals (#4076)
  docs: remove unnecessary spacing element from sales modals (#4075)
  fix: update code examples data manager summary count (#4065)
  fix: update data entry grid example toolbar (#4072)
  fix: remove non-component cell renderer templates from page examples (#4068)
  fix: remove incorrect form control class from input box examples (#4066)
* main:
  fix(components/theme): fix default semantic header styling (#4093)
  feat(components/packages): create schematic to convert SKY UX projects to use standalone components (#4082)
  docs(components/modals): update `wrapperClass` description (#4090)
  fix(components/tabs): tab dropdown close button is on the right in modern theme (#4073)
  fix(components/ag-grid): lookup cell editor should not stop editing when adding a new record (#4086)
  chore(apps/playground): add autonumeric playground (#4088)
  chore: release 13.8.2 (#4087)
  fix(components/layout): use correct background and border for toolbar view actions (#4083)
  fix(components/lookup): use correct mobile search background color, collapse button size (#4085)
  chore: release 13.8.1 (#4081)
  docs(components/ag-grid): lookup properties call out `wrapperClass` requirements for `addClick` (#4084)
  fix: remove redundant header classes  (#4071)
  fix(components/lookup): clicking selection modal label area selects item (#4074)
  fix(components/tiles): use same heading size for tile title and summary (#4077)
  chore: release 13.8.0 (#4079)
* origin/main:
  fix(components/lookup): selection modal show only selected items return known selections (#4101)
  chore: release 13.10.0 (#4099)
  chore(components/theme): add internal eyebrow font class (#4100)
  feat: update ag grid package versions (#4095)
  feat: update design tokens to 3.3.0 (#4098)
  chore: release 13.9.1 (#4097)
  fix: restore module providers (#4096)
  chore: release 13.9.0 (#4091)
  fix(components/filter-bar): add top border styling for filter bar in data manager (#4094)

# Conflicts:
#	libs/components/ag-grid/package.json
* origin/main:
  chore: release 13.11.2 (#4127)
  fix(components/ag-grid): wait for grid to be ready before applying data manager column sizes (#4128)
  fix(components/data-manager): cleanup observable subscription (#4126)
  chore: release 13.11.1 (#4125)
  fix(components/code-examples): simplify data manager code examples (#4122)
  fix(components/docs-tools): allow for code examples without specifying `componentType` (#4124)
  build: code example playground should use up to date examples and dependencies (#4121)
  chore: release 13.11.0 (#4102)
  feat: update design tokens to 4.0.2 (#4106)
  fix(components/code-examples): update record page with block layout to match demo (#4116)
  chore: changelog for 12.44.1 (#4118) 🍒 (#4120)
  fix(components/ag-grid): cell editors should stop editing on focusout event (#4117) (#4119)
  fix: update dependencies to address vulnerabilities (#4113) (#4115)
  chore: changelog for 12.44.0 (#4112) 🍒 (#4114)
  chore: changelog for 12.43.0 (#4107) 🍒 (#4109)
  fix(components/forms): icon checkboxes use correct border radius token in modern theme (#4108)
  fix(components/autonumeric): return correct value when typing on an Android device (#4104) (#4110)

# Conflicts:
#	libs/components/ag-grid/src/lib/modules/ag-grid/ag-grid-data-manager-adapter.directive.ts
* origin/main:
  chore: release 13.11.3 (#4130)
  fix(components/ag-grid): wait for grid to be ready before api call (#4129)
* origin/main:
  build: split manifest and code example builds (#4139)
  chore: release 13.11.4 (#4134)
  chore: update library resources (#4136) (#4138)
  fix(components/modals): add font family override for modal header in default (#4135)
  fix(components/select-field): update menu icon (#4132)
[AB#3605105](https://dev.azure.com/blackbaud/f565481a-7bc9-4083-95d5-4f953da6d499/_workitems/edit/3605105)

- Added generic type parameters to filter interfaces
(`SkyFilterStateFilterValue`, `SkyFilterBarFilterItem`, etc.) for type
safety while maintaining backward compatibility with unknown defaults
- Implemented external filtering logic in `SkyAgGridComponent`
supporting text, number, date, boolean filters with configurable
operators
- Created utility functions (`getFilterValue`, `hasFilterValue`,
`createFilterItem`) for type-safe filter management
…refactor data-grid-filter.spec.ts away from using 'any' so frequently
* origin/13.x.x:
  build: create routes for code examples (#4173)

# Conflicts:
#	libs/components/code-examples/routes/src/index.ts
#	scripts/prebuild-code-examples.mts
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 137 out of 141 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…iewkeeper for header row, create 'pageRequest' output
* origin/13.x.x:
  fix(components/data-manager): set docked data manager to be `inset`  (#4180)
  docs(components/pages): include SkyModalLinkListComponent in docs (#4178)
  feat(components/filter-bar): extend generics for filter types in filter items (#4175)
* origin/13.x.x:
  fix(components/forms): select inputs have proper text overflow and right padding (#4168)
  fix(components/action-bars): default theme summary action bar secondary action button dropdown uses correct button class (#4188)
Copy link
Copy Markdown
Member

@Blackbaud-SteveBrush Blackbaud-SteveBrush left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I resolved the other conversations so we can continue with fresh context. We're closer, but I think we're still looping on the same points. To avoid more back and forth, here's what I'd like to see in the next iteration:

  • Implement loadAsync as the mode switch for async "hard mode"
  • Remove @skyux/data-manager as a peer dependency
  • Replace the experimental resource implementation in the code examples

If you see any blockers or have concerns about these changes that I haven't considered, let's talk through them before the next review.

Comment thread libs/components/data-grid/src/lib/modules/data-grid/data-grid.component.ts Outdated
@Blackbaud-SteveBrush
Copy link
Copy Markdown
Member

Just as a point of record: we talked offline and agreed we need to pivot on this and create two separate components: one to handle static data w/ automatic sorting/etc. and one to handle asynchronous data loading. This will help to keep the APIs and implementations simpler for our consumers.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 9, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch data-grid

Comment @coderabbitai help to get the list of available commands and usage tips.

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

Labels

risk level (author): 1 No additional bugs expected from this change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants