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: add table column setting to Environments page #2789

Merged

Conversation

agatha197
Copy link
Contributor

@agatha197 agatha197 commented Oct 29, 2024

Changes:

This PR enhances the ImageList component in the following ways:

  1. Adds a new TableColumnsSettingModal component for customizing visible columns.
  2. Implements column visibility persistence using local storage.
  3. Introduces a settings button to open the column customization modal.
  4. Improves the digest column rendering with ellipsis and tooltip for better readability.

How to test:

endpoint: 10.100.64.15

  1. Checkout core branch to topic/10-22-feature_add_info_field_to_gql_image_schema
  2. Visit Environments page.
  3. Check the setting icon at the bottom of the table.

Rationale:

These changes provide users with more control over the information displayed in the image list, improving the overall user experience and allowing for a more personalized view of the data.

Effects:

  • Users can now customize which columns are visible in the image list.
  • Column visibility preferences are persisted across sessions.
  • Long digest values are now truncated with an ellipsis and full text available via tooltip.

Screenshots:
image.png

Checklist:

  • Documentation: Update user guide to explain the new column customization feature
  • Test case: Verify that column visibility settings are correctly saved and applied
  • Test case: Ensure the digest column truncation and tooltip work as expected

Copy link

graphite-app bot commented Oct 29, 2024

Your org requires the Graphite merge queue for merging into main

Add the label “flow:merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “flow:hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

Copy link
Contributor Author

agatha197 commented Oct 29, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @agatha197 and the rest of your teammates on Graphite Graphite

@github-actions github-actions bot added the size:M 30~100 LoC label Oct 29, 2024
@agatha197 agatha197 force-pushed the feature/add-AliasedBaseImageName branch from cf42f0e to 09bf733 Compare October 29, 2024 10:54
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from 2b8056f to 1dbe343 Compare October 29, 2024 10:54
@agatha197 agatha197 force-pushed the feature/add-AliasedBaseImageName branch from 09bf733 to a98a494 Compare October 30, 2024 02:03
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch 2 times, most recently from d8c4775 to 412bd1c Compare October 31, 2024 01:28
@agatha197 agatha197 changed the base branch from feature/add-AliasedBaseImageName to feature/replace-name-with-namespace October 31, 2024 01:28
@agatha197 agatha197 force-pushed the feature/replace-name-with-namespace branch from 01fc2fd to 047ba54 Compare October 31, 2024 03:37
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from 412bd1c to dab23a0 Compare October 31, 2024 03:37
@agatha197 agatha197 force-pushed the feature/replace-name-with-namespace branch from 047ba54 to 6695815 Compare October 31, 2024 03:42
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from dab23a0 to 0011774 Compare October 31, 2024 03:42
@agatha197 agatha197 added this to the 24.09 milestone Oct 31, 2024
@agatha197 agatha197 requested a review from lizable October 31, 2024 06:26
@agatha197 agatha197 force-pushed the feature/replace-name-with-namespace branch from 6695815 to cbb0e50 Compare October 31, 2024 08:02
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from 0011774 to fde8595 Compare October 31, 2024 08:02
Copy link
Contributor

@ironAiken2 ironAiken2 left a comment

Choose a reason for hiding this comment

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

LGTM

@agatha197 agatha197 force-pushed the feature/replace-name-with-namespace branch from cbb0e50 to e1139c6 Compare November 1, 2024 03:15
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from fde8595 to ede0978 Compare November 1, 2024 03:15
@agatha197 agatha197 mentioned this pull request Nov 4, 2024
6 tasks
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from 2346c51 to 55bcf7e Compare November 4, 2024 05:02
@agatha197 agatha197 changed the base branch from feature/replace-name-with-namespace to feat/add-getImageFullName-test-file November 4, 2024 05:02
@agatha197 agatha197 force-pushed the feat/add-getImageFullName-test-file branch from b8a0b37 to a980e40 Compare November 4, 2024 05:11
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch 2 times, most recently from 70a122e to 58212b1 Compare November 4, 2024 05:52
@agatha197 agatha197 changed the base branch from feat/add-getImageFullName-test-file to feature/replace-name-with-namespace November 4, 2024 05:52
@agatha197 agatha197 force-pushed the feature/replace-name-with-namespace branch from 92cdd3e to 143e45d Compare November 4, 2024 06:51
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from 58212b1 to c55d292 Compare November 4, 2024 06:51
@agatha197 agatha197 force-pushed the feature/replace-name-with-namespace branch from 143e45d to 2ba59aa Compare November 4, 2024 07:02
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from c55d292 to 23f8a66 Compare November 4, 2024 07:02
@agatha197 agatha197 force-pushed the feature/replace-name-with-namespace branch from 2ba59aa to 7dbc971 Compare November 4, 2024 07:37
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from 23f8a66 to 232381e Compare November 4, 2024 07:37
@agatha197 agatha197 modified the milestones: 24.09, 24.12 Nov 4, 2024
@agatha197 agatha197 force-pushed the feature/replace-name-with-namespace branch 2 times, most recently from 408ee75 to d7d64c4 Compare November 5, 2024 09:55
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from 232381e to 7662c65 Compare November 5, 2024 09:55
@github-actions github-actions bot added the area:lib Library and SDK related issue. label Nov 5, 2024
Copy link

graphite-app bot commented Nov 5, 2024

Merge activity

@agatha197 agatha197 changed the base branch from feature/replace-name-with-namespace to graphite-base/2789 November 5, 2024 10:04
@graphite-app graphite-app bot changed the base branch from graphite-base/2789 to main November 5, 2024 10:06
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from 7662c65 to 53af4e3 Compare November 5, 2024 10:33
**Changes:**

This PR enhances the ImageList component in the following ways:

1. Adds a new TableColumnsSettingModal component for customizing visible columns.
2. Implements column visibility persistence using local storage.
3. Introduces a settings button to open the column customization modal.
4. Improves the digest column rendering with ellipsis and tooltip for better readability.

**How to test:**
> endpoint: 10.100.64.15
1. Checkout core branch to `topic/10-22-feature_add_info_field_to_gql_image_schema`
2. Visit Environments page.
3. Check the setting icon at the bottom of the table.

**Rationale:**

These changes provide users with more control over the information displayed in the image list, improving the overall user experience and allowing for a more personalized view of the data.

**Effects:**

- Users can now customize which columns are visible in the image list.
- Column visibility preferences are persisted across sessions.
- Long digest values are now truncated with an ellipsis and full text available via tooltip.

**Screenshots:**
![image.png](https://graphite-user-uploaded-assets-prod.s3.amazonaws.com/2HueYSdFvL8pOB5mgrUQ/e85e86c3-85c9-4137-9571-d707f6ae4fce.png)

**Checklist:**

- [ ] Documentation: Update user guide to explain the new column customization feature
- [ ] Test case: Verify that column visibility settings are correctly saved and applied
- [ ] Test case: Ensure the digest column truncation and tooltip work as expected
@agatha197 agatha197 force-pushed the feature/add-table-column-setting-to-environment-page branch from 53af4e3 to 813b937 Compare November 5, 2024 10:35
Copy link

github-actions bot commented Nov 5, 2024

Coverage report for ./react

St.
Category Percentage Covered / Total
🔴 Statements
4.92% (-0% 🔻)
349/7096
🔴 Branches
4.54% (-0% 🔻)
219/4828
🔴 Functions
2.85% (-0% 🔻)
67/2350
🔴 Lines
4.81% (-0% 🔻)
334/6942

Test suite run success

93 tests passing in 12 suites.

Report generated by 🧪jest coverage report action from 813b937

@graphite-app graphite-app bot merged commit 813b937 into main Nov 5, 2024
7 checks passed
@graphite-app graphite-app bot deleted the feature/add-table-column-setting-to-environment-page branch November 5, 2024 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:lib Library and SDK related issue. size:M 30~100 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants