security-and-auditing-full-course-s23 icon indicating copy to clipboard operation
security-and-auditing-full-course-s23 copied to clipboard

Published 20 hours ago • verified publisher icon Cyfrin

SmallProxy.sol does not compile in Remix without adding a Receive function or using 'hide warnings'.

Open 0xTrip opened this issue 1 year ago • 1 comments
trafficstars

Lesson

Lesson 1

Could you please leave a link to the timestamp in the video where this error occurs? (You can right click a video and "copy video URL at current time")

https://youtu.be/pUWmJ86X_do?si=xRf3iXmIybjkJTFL&t=6165

Operating System

Windows

Describe the bug

At this part of the video, the student is encouraged to play with SmallProxy.sol from: https://github.com/Cyfrin/foundry-upgrades-f23/blob/main/src/sublesson/SmallProxy.sol

  • A link to this repo could be a good value add, instead of the user searching for it.
  • Upon pasting into Remix, contract will not compile until 'hide warnings' is ticked. It may be worth a small comment to inform the user: '// if pasting into Remix, you might need to tick 'hide warnings'

Great course!

0xTrip avatar Apr 18 '24 12:04 0xTrip

Ah it seems like I was actually also suffering from a Remix/Chrome issue: https://ethereum.stackexchange.com/questions/162507/issue-with-accounts-not-displaying-in-remix-vm-cancun-and-shanghai-environment

You are able to deploy the contracts even with the warnings!

0xTrip avatar Apr 18 '24 12:04 0xTrip

Great! Closing

PatrickAlphaC avatar May 26 '24 18:05 PatrickAlphaC