open-sauced / app

🍕 Insights into your entire open source ecosystem.
https://pizza.new
Apache License 2.0
381 stars 202 forks source link

Feature: hacktoberfest activity graph should only show data from oct 1st - oct 31st #2144

Open jpmcb opened 7 months ago

jpmcb commented 7 months ago

Type of feature

🍕 Feature

Current behavior

The activity chart for Hacktoberfest looks strange since the data returned by the API for the hacktoberfest topic only return data for the configured hacktoberfest year:

Screen Shot 2023-11-14 at 2 23 08 PM

So this chart is technically correct since the API isn't returning data from a few days ago, only data from Oct 1st - Oct 31st.

cc @bdougie noticed this on stream today

Suggested solution

The activity chart should only show data from Oct 1st - Oct 31st when the topic is "hacktoberfest", similar to how the API returns the data.

Additional context

No response

Code of Conduct

Contributing Docs

github-actions[bot] commented 7 months ago

Thanks for the issue, our team will look into it as soon as possible! If you would like to work on this issue, please wait for us to decide if it's ready. The issue will be ready to work on once we remove the "needs triage" label.

To claim an issue that does not have the "needs triage" label, please leave a comment that says ".take". If you have any questions, please reach out to us on Discord or follow up on the issue itself.

For full info on how to contribute, please check out our contributors guide.

takanome-dev commented 7 months ago

The activity chart should only show data from Oct 1st - Oct 31st when the topic is "hacktoberfest", similar to how the API returns the data.

I think the data is correct but there is an overflow happening here probably due to the position of the chart data in the x axis

chart

Maybe we should fix the overflow and change the default topic here.

jpmcb commented 7 months ago

Exactly: the data is correct, it just shouldn't be based on "now", it should be solely based on Oct 1st - Oct 31st when the topic is "hacktoberfest"

a0m0rajab commented 7 months ago

Adding the link: https://beta.app.opensauced.pizza/hacktoberfest/activity/filter/top-100-repos?range=90

and screenshot from today; image

a0m0rajab commented 6 months ago

Related to #2372