search

zzadxz / RATTM

https://rattm.vercel.app
MIT License
0 stars 0 forks source link

Connected CardDataStats to endpoint and added hover effect to AboutYourScore #91

Closed benrockehenderson closed 2 days ago

benrockehenderson commented 2 days ago

Changed CardDataStats.tsx to call the carbon score endpoint every time the component mounts. Updated the props being passed to CardDataStats component in dashboard/page.tsx. Added info icon to CardDataStats component - and added hover state that highlights the AboutCarbonScore component.

https://github.com/user-attachments/assets/c9bd484e-b024-4086-ad60-b3d13e226ec1

vercel[bot] commented 2 days ago

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

Name Status Preview Comments Updated (UTC)
rattm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 22, 2024 11:40pm
zzadxz commented 2 days ago

@benrockehenderson Hey Ben, thank you this is awesome! I was working on fixing the issues in your branch at the same time, so I made my own branch and merged it into yours fixing the problems. It would now deploy!

zzadxz commented 2 days ago

@benrockehenderson Ok I figured the problem. Your local repo was not updated with our app.json for our test scripts on Heroku CI. I'm gonna add them to your commit.

zzadxz commented 2 days ago

I think the branch was behind from main because the tests were passing before that, I'm gonna solve merge conflicts w main now

zzadxz commented 2 days ago

@benrockehenderson Beautiful, all is good now! Thank you!