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 OpenSourceObserver stats to explorer #3344

Merged
merged 16 commits into from
Apr 30, 2024
Merged

Conversation

MasterHW
Copy link
Contributor

@MasterHW MasterHW commented Apr 23, 2024

Fixes: Feature add - show codebase-derived insights to Explorer project page above Karma GAP section. Data provided by Open Source Observer.

Description

Created relevant OSO files in a manner similar to the existing Karma GAP implementation. Should fetch information relevant to project age, number of code contributors, and "velocity" (increasing activity or not) and show them in the UI. If no information can be fetched, show nothing.

Checklist

This PR:

  • Does it add new payout or donation token? In this case, have they been added to the indexer to avoid it to crash?
  • Includes only changes relevant to the original ticket. Significant refactoring needs to be separated.
  • Doesn't disable eslint rules.
  • Doesn't work around the type checker (including but not limited to: type casts, non-null assertions, @ts-ignore, unjustified optional values).
  • Doesn't contain commented out code.
  • If adding/updating a feature, it adds/updates its test script on Notion.

Copy link

vercel bot commented Apr 23, 2024

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

Name Status Preview Comments Updated (UTC)
builder ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 10:27pm
builder-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 10:27pm
explorer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 10:27pm
explorer-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 10:27pm
manager ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 10:27pm
manager-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 30, 2024 10:27pm

Copy link

sentry-io bot commented Apr 23, 2024

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: packages/grant-explorer/src/features/round/ViewProjectDetails.tsx

Function Unhandled Issue
ViewProjectDetails TypeError: Cannot read properties of null (reading 'metadata') mapApplicationToProject(features/projects/hooks/u...
Event Count: 6 Affected Users: 7

Did you find this useful? React with a 👍 or 👎

@MasterHW
Copy link
Contributor Author

Failing because Error: OpenSourceObserver API key not set., which is expected behaviour if no REACT_APP_OSO_API_KEY. Where can I set this for the job?

@ryscheng
Copy link

This is so cool!!
Thank you for putting this together!

I just wanted to give you a heads up that we're trying to get to v1 of our marts models:
That means you should start to see _v1, _v2 style major versioning in the next set of data pipeline runs. We're also improving our style guide for model naming so folks know what to expect in the future. Thanks for being patient with us as we steady these models. We'll aim to use this versioning to avoid major breakages in the future.

But for this iteration, we apologize if this breaks any of your integrations. Please see our GraphQL explorer to see the current status of the model names
https://cloud.hasura.io/public/graphiql?endpoint=https://opensource-observer.hasura.app/v1/grap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog issues we pointed but later scoped out/de-prioritized explorer
Projects
Status: QA
Development

Successfully merging this pull request may close these issues.

Add OpenSourceObserver stats to explorer project page
5 participants