Skip to content

feat: add storage group page - #1289

Merged
astandrik merged 9 commits into
mainfrom
astandrik.add-storage-group-page-1159
Sep 17, 2024
Merged

astandrik merged 9 commits into
mainfrom
astandrik.add-storage-group-page-1159

Conversation

@astandrik

@astandrik astandrik commented Sep 12, 2024 •

Copy link
Copy Markdown
Contributor

Stand

Closes #1159

CI Results

Test Status: ⚠️ FLAKY

📊 Full Report

Total Passed Failed Flaky Skipped
124 116 0 8 0

Bundle Size: 🔺

Current: 78.96 MB | Main: 78.84 MB
Diff: +0.12 MB (0.15%)

⚠️ Bundle size increased. Please review.

ℹ️ CI Information
  • Test recordings for failed tests are available in the full report.
  • Bundle size is measured for the entire 'dist' directory.
  • 📊 indicates links to detailed reports.
  • 🔺 indicates increase, 🔽 decrease, and ✅ no change in bundle size.

Comment thread src/containers/StorageGroupPage/StorageGroupPage.scss
Comment thread src/components/StorageGroupInfo/i18n/en.json Outdated
Comment thread src/components/StorageGroupInfo/i18n/en.json Outdated
Comment thread src/components/StorageGroupInfo/i18n/en.json Outdated
@astandrik
astandrik force-pushed the astandrik.add-storage-group-page-1159 branch from 0f71f28 to 6e0eeb4 Compare September 16, 2024 12:53
@Raubzeug

Copy link
Copy Markdown
Contributor

A bug when scrolling:

Screenshot 2024-09-17 at 09 59 08

if (valueIsDefined(LatencyPutTabletLog)) {
storageGroupInfoSecondColumn.push({
label: storageGroupInfoKeyset('latency-put-tablet-log'),
value: `${LatencyPutTabletLog} ms`,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Here and farther lets add spaces between the digits of the number (like it is done in Available Space field)

}

return (
<div className={b('wrapper', className)}>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lets use Flex component


return (
<div className={b('wrapper', className)}>
<div className={b('col')}>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

do we need this wrapper?

};

return (
<div className={storageGroupPageCn()}>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Flex?

@astandrik
astandrik requested a review from Raubzeug September 17, 2024 11:28
@astandrik
astandrik merged commit 8e05166 into main Sep 17, 2024
@astandrik
astandrik deleted the astandrik.add-storage-group-page-1159 branch September 17, 2024 17:36
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.

add storage group page

3 participants