F
Flow14mo ago
Giovanni S

NFTStorefront stable cadence tests failing

I'm working on stable cadence preview updates in NFTStorefront and am getting failure in NFTStorefront go tests - https://github.com/onflow/nft-storefront/pull/89 Anyone able to help troubleshoot? (fair warning: I'm new to go)
18 Replies
Needle
Needle14mo ago
I've created a thread for your message. Please continue any relevant discussion in this thread. You can rename this thread using /title <new title> If this is a technical question that others may benefit from, considering also asking it on Stackoverflow: https://stackoverflow.com/questions/ask?tags=onflow-cadence
Unknown User
Unknown User14mo ago
Message Not Public
Sign In & Join Server To View
Giovanni S
Giovanni SOP14mo ago
I ran that locally in both lib/go/contracts and lib/go/tests and there's no diff in go.sum, should there be?
Unknown User
Unknown User14mo ago
Message Not Public
Sign In & Join Server To View
Giovanni S
Giovanni SOP14mo ago
When I run make ci locally from root, I get build failure on github.com/onflow/kitty-items/lib/go/test. Is it pulling the tests from kitty-items?
Unknown User
Unknown User14mo ago
Message Not Public
Sign In & Join Server To View
Giovanni S
Giovanni SOP14mo ago
Thank you! Yeah a couple, esp NFTCatalog related need updates (pending updates to NFTCatalog). Best way to get your changes to my pr branch prob a pr to this base yeah? https://github.com/onflow/nft-storefront/tree/stable-cadence-refactor
Unknown User
Unknown User14mo ago
Message Not Public
Sign In & Join Server To View
Giovanni S
Giovanni SOP14mo ago
I'm assuming this feature branch does: https://github.com/onflow/sdks/tree/feature/stable-cadence cc: @Supun
Unknown User
Unknown User14mo ago
Message Not Public
Sign In & Join Server To View
Giovanni S
Giovanni SOP14mo ago
nvm just merged your branch to mine locally I worked off of this branch's contents. https://github.com/onflow/flow-nft/tree/standard-v2 Are these repos being imported as go modules? BTW looks like your emulator updates did it. Thank you!
Unknown User
Unknown User14mo ago
Message Not Public
Sign In & Join Server To View
Giovanni S
Giovanni SOP14mo ago
Ah I see. I thought the transactions had been updated when the contracts were. Alright, I'll need to sort out the dependency repos before continuing on with the NFTStorefront PR then Weird, how is it working under the hood in emulator then?
Unknown User
Unknown User14mo ago
Message Not Public
Sign In & Join Server To View
Giovanni S
Giovanni SOP14mo ago
Thanks for digging into that, I should've looked at those repos first as well
Unknown User
Unknown User14mo ago
Message Not Public
Sign In & Join Server To View
Giovanni S
Giovanni SOP14mo ago
Will do, thanks again!
Unknown User
Unknown User14mo ago
Message Not Public
Sign In & Join Server To View
Want results from more Discord servers?
Add your server