Fund sidebar: power card + recently visited - #982
Conversation
Introduce FundSidebar with funding power stats and recently visited suggestions so the homepage hub can mount a shared right rail. Co-authored-by: Cursor <cursoragent@cursor.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a3ca7bc2fd
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| * Shared Fund right column (Activity / RFPs / Proposals): funding power on top, | ||
| * recently visited beneath (dropped entirely once cleared / empty). | ||
| */ | ||
| export function FundSidebar() { |
There was a problem hiding this comment.
As of this commit, rg "FundSidebar" only finds this declaration and imports inside the same file; /fund, /fund/proposals, /grants, and the grant layout still pass FundingSidebarServer while /activity sets rightSidebar={false}. That means the funding power and recently visited widgets never render in any of the Activity/RFP/Proposal surfaces described here, so this ships as dead UI unless the existing right-sidebar wiring is replaced or composed with this component.
Useful? React with 👍 / 👎.
|
|
||
| // Wait for auth + (when showing USD) a usable rate so we don't flash the | ||
| // empty CTA or an RSC figure that then swaps to dollars. | ||
| const isReady = !isUserLoading && (!showUSD || (!isRateLoading && exchangeRate > 0)); |
There was a problem hiding this comment.
When USD is selected, which is the default preference, a failed exchange-rate fetch leaves isRateLoading false and exchangeRate at 0 in ExchangeRateProvider; this condition therefore never becomes ready and the card stays in the skeleton forever, even for signed-out users or for users whose RSC balance could still be shown. Please stop requiring exchangeRate > 0 after loading completes, or fall back to the RSC/empty state on rate errors.
Useful? React with 👍 / 👎.
…ecessary comments in FundSidebar.
|
1 participant
Summary
FundSidebarwith funding power stats and recently visited entriesStack