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

Add stats bar to transparency page #406

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

moteanup24
Copy link
Contributor

@moteanup24 moteanup24 commented Feb 8, 2025

Build preview:


@dergigi

with apologies I somehow messed up my other branch during my efforts to rebase/update with the upstream master so I'm opening a new PR

this update include formatting changes to the widget (eg it previously had centering issues)

I am still working through using service account authentication. My previous effort to use getStaticProps does not appear to be fruitful as it seems the function is reserved for pages rather than components. So I'm looking further into that.

Copy link

vercel bot commented Feb 8, 2025

@moteanup24 is attempting to deploy a commit to the OpenSats Team on Vercel.

A member of the Team first needs to authorize it.

@dergigi dergigi changed the title create stats widget to display on transparency page Add stats bar to transparency page Feb 9, 2025
Copy link

vercel bot commented Feb 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
os-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 9, 2025 10:24pm

@moteanup24
Copy link
Contributor Author

@dergigi
This iteration uses the service account authentication. It works on my end on a test data file. Pls let me know of any comments!
You'll notice StatsBar is less self-contained here, and is called outside of the MDX rendering again. This is because it seemed natural to fetch the data on the server side using getStaticProps, and I haven't figured out how to propagate props through to a component inside the MDX rendering. But I wanted to start with something that works and uses service account authentication!

Pls also note the SCOPES question--my understanding is that this is scoped to provide read-only access to only the specified file.

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.

1 participant