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

chore: In gnodev serveGnoWebServer, set webConfig HelpChainID and HelpRemote #1497

Merged
merged 1 commit into from
Jan 11, 2024

Conversation

jefft0
Copy link
Contributor

@jefft0 jefft0 commented Jan 5, 2024

In gnodev, serveGnoWebServer sets up the webConfig. Here we set HelpChainID and HelpRemote to the same values that are printed when gnodev starts. Now these appear in gnoweb in "-chain-id" and "-remote" on the help pages as shown below.

NOTE: This PR depends on the fix in PR #1496 .

gnokey maketx call -pkgpath "gno.land/r/demo/boards" -func "GetBoardIDFromName"
-gas-fee 1000000ugnot -gas-wanted 2000000 -send "" -broadcast -chainid "tendermint_test" -args ""
-remote "tcp://0.0.0.0:36657" ADDRESS

…pRemote

Signed-off-by: Jeff Thompson <jeff@thefirst.org>
@jefft0
Copy link
Contributor Author

jefft0 commented Jan 10, 2024

The dependency PR #1496 is merged.

Copy link
Member

@thehowl thehowl left a comment

Choose a reason for hiding this comment

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

LGTM, pinging @gfanton for second review + eventual merge

Copy link
Member

@gfanton gfanton left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@gfanton gfanton merged commit 0a2ba11 into gnolang:master Jan 11, 2024
8 checks passed
gfanton pushed a commit to moul/gno that referenced this pull request Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Status: No status
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants