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

[TSK-259] Create a List UI component ( similar to Notion) that wraps around AG Grid #8

Merged
merged 16 commits into from
Aug 11, 2024

Conversation

alebg
Copy link
Contributor

@alebg alebg commented Aug 3, 2024

No description provided.

carlapbg and others added 16 commits August 3, 2024 15:32
- Create test for LoginCard
- Adapt test for ConversationDialog
…ith pagination and controls

- Todo: styling, filtering
- todo: test how selected items are handled
  + E.g., one page will need batch download; another one will need the selection to create a conversation
- todo: improve actions
  + Download
  + Copy relative path
  + Copy name
- Todo: code, apart from this, a simple upload button that will be combined with the table for one of the pages
…reates a button that executes the function on a list of selected rows

- Also, reorganized Story paths
…+ callBack functions as inputs, and renders them in the top div

- Improved style and allowed for dark mode of top div
- TODO: AGGrid theming
- Theming had to be hardcoded in the AG Grid way (with a css file, generated by their official theme builder), couldn't make it work with tailwind. NOTE: it's always in dark mode
- Similarly, height and width of the AG Grid are not straightforwardly manipulated, so they have to be more or less hardcoded in the AG Grid variants (Source Data and Conversations) depending on the amount of columns
@alebg alebg requested a review from maany August 3, 2024 13:35
@alebg alebg self-assigned this Aug 3, 2024
Copy link

Copy link

github-actions bot commented Aug 3, 2024

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 43.36% (🎯 10%) 1043 / 2405
🟢 Statements 43.36% (🎯 10%) 1043 / 2405
🟢 Functions 33.33% (🎯 10%) 10 / 30
🟢 Branches 59.64% (🎯 10%) 34 / 57
File CoverageNo changed files found.
Generated in workflow #20

@Deeptikandhol10
Copy link

@manny, could you review it?

@maany maany merged commit 02e8417 into dream-aim-deliver:main Aug 11, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants