Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove override keyword. code now compiles. #12

Open
wants to merge 1 commit into
base: 2-workshop-with-gsn
Choose a base branch
from

Conversation

m-waqas88
Copy link

When compiling, the below error was produced

CompileError: TypeError: Public state variable has override specified but does not override anything.

I removed the override specifier and then compiled the code and did test the overall execution. It now runs fine as expected.

@@ -15,7 +15,7 @@ contract CaptureTheFlag is ERC2771Recipient {
_setTrustedForwarder(forwarder);
}

string public override versionRecipient = "3.0.0";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better remove it completely, since it is unused (was in use in previous version)
however, we keep this repo clean, with a single commit for each step, so we will have to re-build it with this change.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@drortirosh great suggestion. I do agree with you. Build a clean repo. I want to contribute to this and do this for you guys. What would be the way forward?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@drortirosh please note that this is a "generated" branch - please recreate the workshop for beta.3 when you have the time! 😀

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@forshtat I also want to contribute to this recreation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants