Is there some missing context here? It sounds like @rohan did some testing and things didn't work, but not sure what that testing was

In D10419#302273, @ashoat wrote:

Is there some missing context here? It sounds like @rohan did some testing and things didn't work, but not sure what that testing was

Ah sorry I'll add the context here. @ginsu and I spoke offline - I patched in the stack locally and changed the providers to use goerli instead of mainnet (since I have some ENS names linked to wallet addresses on the testnet).

I verified that the ENS names for my users resolved in my inbox, chat, etc, but when I tried hovering over the reaction tooltip, it only showed the wallet addresses. One thing I noticed was when the useENSNames hook was called in the reaction tooltip, ensCache was always undefined so we'd early return. I wasn't sure if that was the cause for them not resolving in the reaction tooltip for me.

Edit: Not sure if it's helpful. but when I move the AlchemyENSCacheProvider above TooltipProvider in app.react.js, the ENS names are resolved now

Edit: Not sure if it's helpful. but when I move the AlchemyENSCacheProvider above TooltipProvider in app.react.js, the ENS names are resolved now

This is super helpful @rohan! Thanks for taking a look at this! I will update the diff once I can properly sign in w/ eth and better test this out myself. (I actually managed to resolve a lot of my metamask linking issues today)

Is this working & ready for review now?

Is this working & ready for review now?

Yes, updated the diff, test plan + reviewers (previously had rohan but he's no longer here) to reflect the new changes I made