Overview - Skyux

skyux

7 days30 days

Latest CI Pipeline Executions

Fuzzy

Succeeded
main
77198c8d fix(components/code-examples): simplify data manager code examples (#4122)
2 days ago by johnhwhite
Succeeded
data-manager-code-example-updates
780643fd Fix options, further simplifications
2 days ago by johnhwhite
Succeeded
4122
a4cd8a3f Merge remote-tracking branch 'origin/main' into data-manager-code-example-updates * origin/main: fix(components/docs-tools): allow for code examples without specifying `componentType` (#4124)
2 days ago by johnhwhite
Succeeded
data-manager-code-example-updates
780643fd Include fixes for data-manager-adapter directive, additional cleanup
2 days ago by johnhwhite
Succeeded
4122
0bf66b7c Include fixes for data-manager-adapter directive, additional cleanup
2 days ago by johnhwhite
Succeeded
data-manager-code-example-updates
ecb38d48 Additional code example changes, allow template signal for cell type
3 days ago by johnhwhite
Failed
4122
e8cb04bd Additional code example changes, allow template signal for cell type
3 days ago by johnhwhite
Succeeded
data-manager-code-example-updates
ecb38d48 Simplify data-grid, data-entry-grid, and data-manager code examples (#4123) ## Summary: Simplify data-grid, data-entry-grid, and data-manager code examples All tasks completed successfully! ✅ ### Changes Made: #### 1. Data-Entry-Grid Examples - **basic/**: Converted to use signals (`signal<AgGridDemoRow[]>`) and `structuredClone()` for modal data passing - Removed unnecessary `refreshCells()` API call - Removed unnecessary `markForCheck()` call - **inline-help/**: Converted to use signals and `structuredClone()` for modal data passing - Removed unnecessary `refreshCells()` API call - Removed unnecessary `markForCheck()` call - **focus/**: Already simplified (reviewed, no changes needed) - **data-manager-added/**: Already uses signals and `structuredClone()` (reviewed, no changes needed) #### 2. Data-Grid Examples - **data-manager/**: Removed `OnInit` and `OnDestroy` lifecycle hooks, simplified to use constructor-based initialization - Restored explanatory comment about fake endpoint in `onJobTitleSearchAsync()` - **data-manager-multiselect/**: Removed `OnInit` and `OnDestroy` lifecycle hooks, simplified to use constructor-based initialization - Restored explanatory comment about fake endpoint in `onJobTitleSearchAsync()` - **paging/**: Converted from manual subscription management to `toSignal()` and `effect()`, removed `OnInit` and `OnDestroy` - Removed unnecessary `markForCheck()` calls - Removed unused `ChangeDetectorRef` import and injection #### 3. Data-Manager Examples - **basic/**: Already using signals properly (reviewed, no changes needed) ### Key Improvements: ✅ All modal contexts now use `structuredClone()` instead of shallow copies ✅ Replaced manual subscription management with signals and effects ✅ Removed unnecessary lifecycle hooks (`OnInit`, `OnDestroy`) where possible ✅ Removed unnecessary `refreshCells()` API calls - signal updates trigger change detection automatically ✅ Removed unnecessary `markForCheck()` calls - signal updates trigger change detection automatically ✅ Restored educational comments explaining limits of example code ✅ Reduced code complexity while maintaining example independence ✅ All examples pass linting ✅ Build completes successfully Each example remains independent and self-contained as required.   <details> <summary>Original prompt</summary> > In PR #4122 I'm modifying some of the code examples to make them less complex. In the code-examples project, each directory with an example.component.ts file should stand as an independent code example. I'd like to apply a similar set of simplifications to the other data-grid, data-entry-grid, and data-manager code examples. Convert to using signals where possible. Fix the edit contexts for modals to use a structuredClone of the original data rather than a shallow copy. Within each example, look for ways to cut down on redundant code while keeping each code example independent of other examples. </details>  --- ✨ Let Copilot coding agent [set things up for you](https://github.com/blackbaud/skyux/issues/new?title=✨+Set+up+Copilot+instructions&body=Configure%20instructions%20for%20this%20repository%20as%20documented%20in%20%5BBest%20practices%20for%20Copilot%20coding%20agent%20in%20your%20repository%5D%28https://gh.io/copilot-coding-agent-tips%29%2E%0A%0A%3COnboard%20this%20repo%3E&assignees=copilot) — coding agent works faster and does higher quality work when set up for your repo. --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: johnhwhite <750350+johnhwhite@users.noreply.github.com>
3 days ago by johnhwhite
Succeeded
data-manager-code-example-updates
ecb38d48 fix(components/code-examples): simplify data manager code examples
3 days ago by johnhwhite
Succeeded
4122
5a1cd348 fix(components/code-examples): simplify data manager code examples
3 days ago by johnhwhite
Succeeded
main
ecb38d48 build: code example playground should use up to date examples and dependencies (#4121) [AB#3647534](https://dev.azure.com/blackbaud/f565481a-7bc9-4083-95d5-4f953da6d499/_workitems/edit/3647534)
4 days ago by johnhwhite
Succeeded
code-examples-affect-manifest
18daf4a3 build: code example playground should use up to date examples and dependencies
4 days ago by johnhwhite
Succeeded
4121
ce6ae3ae build: code example playground should use up to date examples and dependencies
4 days ago by johnhwhite
Succeeded
4119
7b769dd3 fix(components/ag-grid): cell editors should stop editing on focusout event (#4117) [AB#3644316](https://dev.azure.com/blackbaud/f565481a-7bc9-4083-95d5-4f953da6d499/_workitems/edit/3644316)
5 days ago by johnhwhite
Succeeded
12.x.x
94750584 fix(components/ag-grid): cell editors should stop editing on focusout event (#4117) [AB#3644316](https://dev.azure.com/blackbaud/f565481a-7bc9-4083-95d5-4f953da6d499/_workitems/edit/3644316)
5 days ago by johnhwhite
Succeeded
cell-editors-handle-focusout
91743d39 fix(components/ag-grid): cell editors should stop editing on focusout event
5 days ago by johnhwhite
Succeeded
4117
cfc06993 fix(components/ag-grid): cell editors should stop editing on focusout event
5 days ago by johnhwhite
Succeeded
main
91743d39 fix: update dependencies to address vulnerabilities (#4113) (#4115) 🍒 #4113
6 days ago by johnhwhite
Succeeded
4115
9ec6b2f5 Merge remote-tracking branch 'origin/main' into cherry-pick_2c4a81bf1_1765217139755 * origin/main: chore: changelog for 12.44.0 (#4112) 🍒 (#4114)
6 days ago by johnhwhite
Succeeded
cherry-pick_2c4a81bf1_1765217139755
d40764e2 Update snapshots
6 days ago by johnhwhite