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

chore: add dashboard automation #86

Merged
merged 20 commits into from
Jan 31, 2023
Merged

chore: add dashboard automation #86

merged 20 commits into from
Jan 31, 2023

Conversation

SgtPooki
Copy link
Member

@SgtPooki SgtPooki commented Jan 30, 2023

This PR adds reporting capabilities using google spreadsheets. We download data from countly, upload to google spreadsheets, and then use the charts from that data in Notion and Google slides.

The github action included in this PR runs every two hours to keep our metrics up to date (I could easily be swayed to update this to once/twice per day)

fixes ipfs/ipfs-gui#126

@SgtPooki SgtPooki changed the title Feat/reports chore: Create dashboard automation Jan 30, 2023
@SgtPooki SgtPooki changed the title chore: Create dashboard automation chore: add dashboard automation Jan 30, 2023
@SgtPooki SgtPooki changed the title chore: add dashboard automation [WIP] chore: add dashboard automation Jan 30, 2023
@SgtPooki
Copy link
Member Author

successful Github CI action publishing data for us: https://github.com/ipfs-shipyard/ignite-metrics/actions/runs/4049563926/jobs/6966074792

Copy link
Member Author

@SgtPooki SgtPooki left a comment

Choose a reason for hiding this comment

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

self review. update coming.

@SgtPooki SgtPooki changed the title [WIP] chore: add dashboard automation chore: add dashboard automation Jan 31, 2023
@SgtPooki SgtPooki marked this pull request as ready for review January 31, 2023 01:01
Copy link
Collaborator

@whizzzkid whizzzkid left a comment

Choose a reason for hiding this comment

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

lgtm in principle, nothing blocking, some nits and performance improvements, optional.

.github/workflows/metrics-update.yml Show resolved Hide resolved
reports/README.md Show resolved Hide resolved
reports/constants.ts Outdated Show resolved Hide resolved
reports/constants.ts Outdated Show resolved Hide resolved
reports/constants.ts Show resolved Hide resolved
reports/doCountlyFetch.ts Outdated Show resolved Hide resolved
reports/doCountlyFetch.ts Outdated Show resolved Hide resolved
reports/downloadDashboardData.ts Outdated Show resolved Hide resolved

// output the name of the app as row headers and the date labels as column headers
if (!printedHeaders) {
const dateLabels = activeUserData.map((day: any) => new Date(day._id).toISOString().split('T')[0])
Copy link
Collaborator

Choose a reason for hiding this comment

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

smart, might wanna add a comment why this works. just for our futureselves.

Copy link
Member Author

Choose a reason for hiding this comment

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

not sure what we're looking to comment about?

reports/downloadDashboardData.ts Outdated Show resolved Hide resolved
SgtPooki and others added 4 commits January 30, 2023 21:19
Co-authored-by: Nishant Arora <1895906+whizzzkid@users.noreply.github.com>
Co-authored-by: Nishant Arora <1895906+whizzzkid@users.noreply.github.com>
@SgtPooki SgtPooki merged commit b8c0416 into main Jan 31, 2023
@SgtPooki SgtPooki deleted the feat/reports branch January 31, 2023 18:47
github-actions bot pushed a commit that referenced this pull request Jul 13, 2023
## [2.0.0](v1.3.0...v2.0.0) (2023-07-13)

### ⚠ BREAKING CHANGES

* **stores:** Since stores are now async, upgrading to this might require some logic rework.

### Features

* **stores:** Async Stores Release Please ([#132](#132)) ([c12e6ca](c12e6ca))

### Trivial Changes

* add dashboard automation ([#86](#86)) ([b8c0416](b8c0416))
* update docs/telemetry/PRIVACY_POLICY.md ([#76](#76)) ([3321ff1](3321ff1))
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.

Create and document process for generating all-hands reports
2 participants