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.

Sign up for GitHub

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

Jump to bottom

feat: Resource Browser Visualizations #2164

Merged
merged 30 commits into from
Nov 7, 2024
Merged

feat: Resource Browser Visualizations #2164

merged 30 commits into from
Nov 7, 2024

Conversation

RohitRaj011
Copy link
Contributor

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Checklist:

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

@RohitRaj011 RohitRaj011 self-assigned this Nov 4, 2024
@github-actions github-actions bot added the PR:Issue-verification-failed PR:Issue-verification-failed label Nov 4, 2024
eshankvaish
eshankvaish requested changes Nov 4, 2024
View reviewed changes
src/components/cluster/ClusterForm.tsx Outdated Show resolved Hide resolved
src/components/ClusterNodes/types.ts Outdated Show resolved Hide resolved
src/components/ClusterNodes/constants.ts Outdated Show resolved Hide resolved
src/Pages/ResourceBrowser/ClusterList/ClusterMap/utils.ts Outdated Show resolved Hide resolved
src/Pages/ResourceBrowser/ClusterList/ClusterMap/ClusterMap.tsx Show resolved Hide resolved
src/components/ClusterNodes/ClusterSelectionList.tsx Outdated Show resolved Hide resolved
eshankvaish
eshankvaish previously approved these changes Nov 4, 2024
View reviewed changes
@RohitRaj011 RohitRaj011 changed the base branch from main to kubecon-2024 November 5, 2024 17:24
@RohitRaj011
feat: ResourceBrowser - Cluster Data Visualizations - TreeMap, Filter…
8a234d5 
…s based on Status
@RohitRaj011
refactor: ClusterMap - scss update
9e49420
@RohitRaj011
refactor: ClusterMap, ClusterSelectionList - UAT changes
ef58eb4
@RohitRaj011
feat: ClusterMap - add loading state
8e74fd7
@RohitRaj011
fix: ClusterMap - bar padding
b371169
@RohitRaj011
refactor: ClusterMap - Tooltip design update, code refactor
55eff4a
@RohitRaj011
refactor: ClusterSelectionList, ResourceBrowser - code refactor, scro…
d775fd1 
…ll coming even when not overflowing fix
@RohitRaj011
refactor: ClusterSelectionList - Remove table header in empty state
8dcdd6a
@RohitRaj011
feat: Cluster - add Production/Non-Production tag
13e4b28
@RohitRaj011
fix: ClusterMap - tooltip padding
fb87aa8
@RohitRaj011
fix: ClusterSelector - null check missing
fb6a51e
@RohitRaj011
refactor - ClusterSelectionList, ClusterMap - UAT changes
150cc46
@RohitRaj011
feat: ClusterSelectionList - clusterFilter move from state to url
47dba14
@RohitRaj011
refactor: ClusterForm, ClusterSelectionList, ClusterMap - types refactor
98177d7
@RohitRaj011
fix: ClusterForm - types fix
d80971f
@RohitRaj011 RohitRaj011 force-pushed the feat/resource-browser-visualizations branch from ef66d23 to d80971f Compare November 5, 2024 17:26
AbhishekA1509 and others added 14 commits November 6, 2024 11:26
@AbhishekA1509
feat: add MonitoringDashboard
cf6854c
@AbhishekA1509
Update devtron-fe-common-lib to version 0.6.0-beta-1
65526aa
@AbhishekA1509
fix: self review
0e5b3b9
@AbhishekA1509
fix: move fixedTabIndices to util
1a7864c
@AbhishekA1509
Merge pull request #2162 from devtron-labs/feat/rb-monitoring
75821e9 
feat: rb monitoring
@RohitRaj011
feat: ResourceBrowser - hide virtual clusters from page header cluste…
4d7f81c 
…r search, code refactor
@RohitRaj011
refactor: css update
a4dec04
@RohitRaj011
Merge branch 'kubecon-2024' of github.com:devtron-labs/dashboard into…
c208194 
… feat/resource-browser-visualizations
@RohitRaj011
chore: common-lib version bump
319d0b3
@RohitRaj011
fix: ClusterSelectionList - clearFilters not working fix
f3c752c
@RohitRaj011
Merge branch 'kubecon-2024' of github.com:devtron-labs/dashboard into…
f09f7e3 
… feat/resource-browser-visualizations
@RohitRaj011
chore: common-lib version bump
6b51b39
@RohitRaj011
fix: ClusterMap - css update
3f38460
@RohitRaj011
refactor: DynamicTabs - css update
caa16e8
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Nov 7, 2024

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@RohitRaj011 RohitRaj011 merged commit 6849321 into kubecon-2024 Nov 7, 2024
4 of 5 checks passed
@RohitRaj011 RohitRaj011 deleted the feat/resource-browser-visualizations branch November 7, 2024 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vivek-devtron vivek-devtron vivek-devtron approved these changes

@eshankvaish eshankvaish eshankvaish left review comments

@AbhishekA1509 AbhishekA1509 Awaiting requested review from AbhishekA1509

@vikramdevtron vikramdevtron Awaiting requested review from vikramdevtron vikramdevtron is a code owner

Assignees

@RohitRaj011 RohitRaj011

Labels
PR:Issue-verification-failed PR:Issue-verification-failed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@RohitRaj011 @eshankvaish @vivek-devtron @AbhishekA1509