-
Notifications
You must be signed in to change notification settings - Fork 729
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
🚀 Feature: add filtering to the analytics page #147
Comments
Hi @nevo-david I am pretty new to open source configuration and was wondering if I could help out on this issue? |
Hi @jyo142 :) |
@nevo-david thank you for your response. I believe that I have successfully run the project locally. Does this look correct? |
Awesome! |
Hi @nevo-david this may be a silly question... but do you know how I can get past the billing? Whenver I go to the Billing page locally the popup of "Payment Required" keeps showing up even if I click on the "Move to Billing" button. I could play around with the code to try and make it work but I just wanted to make sure that I do it the right way. Do I have to signup in the production instance first? Thanks! |
Yup, just remove the Stripe env variables :)
…On Fri, May 3, 2024 at 2:54 AM James Okada ***@***.***> wrote:
Hi @nevo-david <https://github.com/nevo-david> this may be a silly
question... but do you know how I can get past the billing? Whenver I go to
the Billing page locally the popup of "Payment Required" keeps showing up
even if I click on the "Move to Billing" button. I could play around with
the code to try and make it work but I just wanted to make sure that I do
it the right way. Do I have to signup in the production instance first?
Thanks!
Screenshot.2024-05-02.at.12.52.43.PM.png (view on web)
<https://github.com/gitroomhq/gitroom/assets/4103523/540c9d04-7cb8-44dd-a2b6-6aa0469f0de8>
—
Reply to this email directly, view it on GitHub
<#147 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AX32VBTNQL4YYEEISULYP2LZAKKYTAVCNFSM6AAAAABE5JXO5KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJRGQ2DIMJZGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
hi @nevo-david thank you for the response. Unfortunately removing the env variables still did not help :( just curious would you be available for a quick screenshare sometime? I do really want to help out but hopefully I can resolve this issue soon |
Hey @jyo142 check again to run the project |
@jyo142 we have fixed the problem, can you take the new changes from main? |
@nevo-david @joset98 that worked thanks for the fix! Sorry i have been caught up with work and havent been able to put much attention into this. @nevo-david what is your opinion on using external packages for this project? I was thinking about https://www.npmjs.com/package/react-date-range (it says not being maintained, but that was about 2 weeks ago and it still looks very good and popular) |
@jyo142 thank you for the response, I have somewhat missed it. Sorry for that. It uses mantine, feel free to customize another component from it for a range picker, should be easy :) |
🔖 Feature description
On the main analytics page, add a date range picker between dates.
It will filter both the top graph and the amount of stars at the bottom.
🎤 Why is this feature needed ?
Sometimes, we need more focused analytics on specific dates
✌️ How do you aim to achieve this?
🔄️ Additional Information
No response
👀 Have you spent some time to check if this feature request has been raised before?
Are you willing to submit PR?
None
The text was updated successfully, but these errors were encountered: