Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(ui-common): integrate GlideDataGrid into MatrixGrid #2134

Merged
merged 13 commits into from
Sep 12, 2024

Conversation

hdinia
Copy link
Member

@hdinia hdinia commented Sep 5, 2024

No description provided.

@hdinia hdinia self-assigned this Sep 5, 2024
@hdinia hdinia force-pushed the feature/add-matrix-grid branch 3 times, most recently from 7ec1107 to c4dcf74 Compare September 5, 2024 14:24
@hdinia hdinia requested a review from skamril September 10, 2024 16:58
@hdinia hdinia force-pushed the feature/add-matrix-grid branch 4 times, most recently from 1402aaa to 8cbba31 Compare September 12, 2024 14:35
…operations

- Update and align type definitions for matrix-related operations
- Refactor MatrixUpdateDTO to match actual usage in API interactions
- Enhance GridUpdate and related types for better type safety
- Adjust useMatrix hook to utilize updated type definitions
- Modify test cases to reflect new type structures
- Ensure consistency between client-side types and API expectations
@hdinia hdinia force-pushed the feature/add-matrix-grid branch 4 times, most recently from 677e39c to d810cbe Compare September 12, 2024 15:17
skamril
skamril previously approved these changes Sep 12, 2024
webapp/vite.config.ts Show resolved Hide resolved
@hdinia hdinia changed the title feat(ui): integrate GlideDataGrid into MatrixGrid feat(ui-common): integrate GlideDataGrid into MatrixGrid Sep 12, 2024
@hdinia hdinia merged commit a35b0ce into dev Sep 12, 2024
10 checks passed
@hdinia hdinia deleted the feature/add-matrix-grid branch September 12, 2024 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants