Skip to content

feat: Build Stats#105

Open
MaxIsJoe wants to merge 9 commits intounitystation:developfrom
MaxIsJoe:changelog-goodies
Open

feat: Build Stats#105
MaxIsJoe wants to merge 9 commits intounitystation:developfrom
MaxIsJoe:changelog-goodies

Conversation

@MaxIsJoe
Copy link
Contributor

image

Adds a stats section at the top of the page that showcases how many additions, fixes and improvements were made in the past few builds + a card next to it that displays the latest code-scan version.

@vercel
Copy link

vercel bot commented Apr 21, 2025

@MaxIsJoe is attempting to deploy a commit to the unitystation Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Apr 21, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
unitystation-web Ready Ready Preview Mar 3, 2026 3:15am

@corp-0
Copy link
Member

corp-0 commented Apr 21, 2025

move your stats bellow the heading of the page. I will check the components and stuff after work

const textByType: Dict<string> = {
'announcement': 'Announcement',
'weekly': 'Weekly Update',
'weekly': 'Progress Update',
Copy link
Member

Choose a reason for hiding this comment

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

I think you forgot to rebase. This commmit already exists

return (
<Panel>
<h3 className="text-xl font-bold mb-4 text-white">Recent Build Statistics</h3>
<div className="grid grid-cols-1 md:grid-cols-3 gap-4">
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<div className="grid grid-cols-1 md:grid-cols-3 gap-4">
<div className="grid grid-cols-1 lg:grid-cols-3 gap-4">

there is an awkward transition if you test how responsive is this new component where the 3 cols are not big enough for the text

}

const BuildStatsComponent: React.FC<BuildStatsProps> = ({ builds }) => {
const recentBuilds = builds.slice(0, 150);
Copy link
Member

Choose a reason for hiding this comment

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

150 is a big slice, isn't it?

@@ -0,0 +1,68 @@
import React from 'react';
Copy link
Member

Choose a reason for hiding this comment

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

all components should be inside the outer components folder.

I'd say this component is an organism, so place it inside components/organisms/changelog/

@@ -0,0 +1,15 @@
import React, { ReactNode } from 'react';
Copy link
Member

Choose a reason for hiding this comment

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

mocules?

corp-0 and others added 3 commits December 10, 2025 07:49
# [1.13.0](unitystation/unitystation-web@v1.12.0...v1.13.0) (2025-12-10)

### Bug Fixes

* update all deps ([f307964](unitystation@f307964))
* updated vulnerable deps ([5826ba7](unitystation@5826ba7))

### Features

* changes "Weekly Update" pill's text to "Progress update" ([6845ea6](unitystation@6845ea6))
* first iteration of the new ledger page ([1cae21d](unitystation@1cae21d))
* first iteration of the new ledger page ([cd5c42f](unitystation@cd5c42f))
@MaxIsJoe MaxIsJoe force-pushed the changelog-goodies branch from dacc134 to 865c3ea Compare March 3, 2026 02:01
@MaxIsJoe MaxIsJoe force-pushed the changelog-goodies branch from 865c3ea to 7e9013e Compare March 3, 2026 02:11
MaxIsJoe and others added 4 commits March 3, 2026 04:12
# [1.13.0](unitystation/unitystation-web@v1.12.0...v1.13.0) (2025-12-10)

### Bug Fixes

* update all deps ([f307964](unitystation@f307964))
* updated vulnerable deps ([5826ba7](unitystation@5826ba7))

### Features

* changes "Weekly Update" pill's text to "Progress update" ([6845ea6](unitystation@6845ea6))
* first iteration of the new ledger page ([1cae21d](unitystation@1cae21d))
* first iteration of the new ledger page ([cd5c42f](unitystation@cd5c42f))
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.

3 participants