-
Notifications
You must be signed in to change notification settings - Fork 370
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
feat: add Matija's Homepage realm to examples #2916
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments. please check them out. thank you
main std.Address // tony's main address | ||
backup std.Address // backup address |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these two variable's names are seems ambiguous. Could you update their names? like mainAddr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, please see fix in this commit: 16529d3
|
||
func SetAddress(a std.Address) error { | ||
if !a.IsValid() { | ||
return errors.New("config: invalid address") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
error messages are repeated (invalid arress and unauthorized). please make them as constant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, pleas see fix im this commit: 15e15a2
return backup | ||
} | ||
|
||
func SetAddress(a std.Address) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very niptick: consider change parameter name a
to more concrete name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, please see fix here: ef98d67
…ble's names are seems ambiguous. Could you update their names? like mainAddr)
… on PR - error messages are repeated (invalid arress and unauthorized). please make them as constant.)
…mment on PR - very niptick: consider change parameter name a to more concrete name.)
Thank you for your insights @notJoon. I've fixed the issues, please let me know if I can make anything more appropriate. |
Summary
This pull request adds a new realm example to the Gno
examples
repository—Matija's Homepage. It showcases a personal homepage built on the Gno chain where users can interact by voting with GNOT tokens to change the page's color. The more tokens users send, the greater influence they have on the color scheme, providing an interactive and dynamic experience.Key Features
Tools & Technologies
Why this is valuable
This example highlights the possibilities of personal realms on Gno, showing how users can create unique and interactive profiles. It’s a fun and approachable entry point for anyone new to Gno development, while also demonstrating the platform's flexibility and potential for creative expression.