-
Notifications
You must be signed in to change notification settings - Fork 341
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
fix(gnoweb): FaucetURL correct endpoint #1449
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1449 +/- ##
==========================================
+ Coverage 55.87% 56.12% +0.25%
==========================================
Files 430 431 +1
Lines 65618 65860 +242
==========================================
+ Hits 36667 36967 +300
+ Misses 26083 26009 -74
- Partials 2868 2884 +16
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -20,7 +20,7 @@ | |||
{{ end }} | |||
</script> | |||
|
|||
<form method="POST" action="{{ .Data.faucetURL }}"> | |||
<form method="POST" action="{{ .Data.Faucet.faucetURL }}"> |
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.
Side-note: what a horrible templating engine we use. (These kind of errors can normally be caught by html/template if it's using structs under the hood, but here we're using a bunch of map-string-interfaces so no errors are caught...)
Have you tested this works? Reding through the code it shouldn't, this change should:
<form method="POST" action="{{ .Data.Faucet.faucetURL }}"> | |
<form method="POST" action="{{ .Data.Config.FaucetURL }}"> |
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.
OH Damn!
You're half right, i'm using gnolang/faucet, which use different variable name 😮💨
<!-- please provide a detailed description of the changes made in this pull request. --> Following PR https://github.com/gnolang/gno/pull/1444/files This is needed for portal loop <details><summary>Contributors' checklist...</summary> - [ ] Added new tests, or not needed, or not feasible - [ ] Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory - [ ] Updated the official documentation or not needed - [ ] No breaking changes were made, or a `BREAKING CHANGE: xxx` message was included in the description - [ ] Added references to related issues and PRs - [ ] Provided any useful hints for running manual tests - [ ] Added new benchmarks to [generated graphs](https://gnoland.github.io/benchmarks), if any. More info [here](https://github.com/gnolang/gno/blob/master/.benchmarks/README.md). </details>
Following PR https://github.com/gnolang/gno/pull/1444/files
This is needed for portal loop
Contributors' checklist...
BREAKING CHANGE: xxx
message was included in the description