Closed norbertschuler closed 4 months ago
The recent changes involve integrating environmental contributions data, specifically trees donated, into the LeaderBoard
components across different files. This includes adding new types and updating component props to handle this new data, enhancing the feature's interactivity and informational depth.
File Path | Change Summary |
---|---|
pages/sites/[slug]/[locale]/all.tsx |
Added TreesDonated import, state, and effect, removed redirect . |
src/features/common/types/leaderboard.d.ts |
Added TreesDonated type definition. |
src/tenants/planet/LeaderBoard/components/Stats.tsx |
Updated to include TreesDonated in props and logic. |
src/tenants/planet/LeaderBoard/index.tsx |
Added treesDonated to props and conditional rendering logic. |
🐇✨ A hop through code, a leap through trees, We count each leaf, with the greatest of ease. From branch to branch, our data will flow, In LeaderBoards, where green numbers grow. 🌳📈💚
- CodeRabbit 🐰
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
planet-webapp | ✅ Ready (Inspect) | Visit Preview | Apr 30, 2024 3:04pm | |
planet-webapp-multi-tenancy-setup | ✅ Ready (Inspect) | Visit Preview | Apr 30, 2024 3:04pm |
Ok, I'll just keep this as a draft until releases will work normally again.
@mohitb35 What's the data inconsistency issue?
The numbers shared generally was from Q1. vs the query returns number for 1 week ago.
There isn't any issue as such, but we do choose to hardcode certain figures at times, and I wondered if this was one of those cases.
@norbertschuler We can release this now, since www1.plant-for-the-planet.org is now redirected to web.plant-for-the-planet.org, so changes will be visible.
@norbertschuler We can release this now, since www1.plant-for-the-planet.org is now redirected to web.plant-for-the-planet.org, so changes will be visible.
Ok, I added using an env named WEBHOOK_URL which I also added to all deployments at Heroku and Vercel.
Fixes https://planetfoundation.slack.com/archives/C015AEAU75Z/p1714117221663939?thread_ts=1713974028.645419&cid=C015AEAU75Z
Changes in this pull request:
To-Do:
https://automate.plant-for-the-planet.org/webhook/
in the environment like we already did for the Salesforce deplyoment defined asWEBHOOK_URL
?