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

dashboard: Sleepを表示 / Filter部分にカウントを表示 #993

Merged
merged 6 commits into from
Jan 7, 2025

Conversation

motoki317
Copy link
Member

なぜやるか

やったこと

  • Sleep状態が(なぜか)表示できていなさそうだったので、表示した
    • Idleと見分けるため、色は意図的に変えている 他と被らない紫に適当にした
  • Filter部分にアプリとリポジトリ数のカウントを表示

image

image

変なところがあれば教えてもらえると助かります 🙇

やらなかったこと

資料

Copy link

github-actions bot commented Jan 7, 2025

Preview (prod backend + PR dashboard) → https://993.ns-preview.trapti.tech/

@motoki317 motoki317 force-pushed the dashboard/impr-status branch from 2ed5a18 to 4befd1a Compare January 7, 2025 10:52
@motoki317 motoki317 requested a review from cp-20 January 7, 2025 10:52
@motoki317 motoki317 force-pushed the dashboard/impr-status branch from 4befd1a to 6365f94 Compare January 7, 2025 10:53
@motoki317 motoki317 force-pushed the dashboard/impr-status branch from 6365f94 to f5a4a26 Compare January 7, 2025 10:54
Copy link
Contributor

@cp-20 cp-20 left a comment

Choose a reason for hiding this comment

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

実装ありがとうございます! ちゃんと動いてて全体的には良さそうです!

Comment on lines 186 to 191
const filteredReposByOrigin = () => {
return repos.filter((r) => origins.includes(repositoryURLToOrigin(r.url))) ?? []
}
const filteredApps = () => {
return apps.filter((a) => statuses.includes(applicationState(a))) ?? []
}
Copy link
Contributor

Choose a reason for hiding this comment

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

細かいですが、ここをわざわざ関数にする必要はない気がします

@motoki317 motoki317 merged commit 136bb2b into main Jan 7, 2025
9 checks passed
@motoki317 motoki317 deleted the dashboard/impr-status branch January 7, 2025 13:51
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.

2 participants