You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have to do a little more research, but I believe if you add the constant keyword to a function it makes it so that it can only be executed via a readonly call instead of a writeable transaction. This is important, since some of these currently call out to the external Asset contract, and if we make the getOwner() function constant as well in the interface, then users can trust this function regardless of the implementation since they won't be able to update any state.
The text was updated successfully, but these errors were encountered:
While recognizing I'm a bit late here, in the latest release of solidity >= 0.5.xx, this is achieved via the pure and view function modifiers.
I've since updated AuctionHouse.sol utilizing these modifiers and sucessfully migrating, however I've been itching to play with Trinity and Web3.py vs geth ahead of Ethereum 2.0 and ERC 1155
I'm noticing a bit of overlap with a few of the issues. I'll submit my PR here shortly.
I have to do a little more research, but I believe if you add the constant keyword to a function it makes it so that it can only be executed via a readonly call instead of a writeable transaction. This is important, since some of these currently call out to the external Asset contract, and if we make the
getOwner()
function constant as well in the interface, then users can trust this function regardless of the implementation since they won't be able to update any state.The text was updated successfully, but these errors were encountered: