-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Merge Params with dictionary #6633
Comments
No, that would not make sens. |
@bep Thank you, such fix is also fine :) |
bep
added a commit
to bep/hugo
that referenced
this issue
Dec 19, 2019
* If the end result is maps.Params, we need to lower the keys * Also make sure to handle nested maps of different type Update gohugoio#6633
bep
added a commit
that referenced
this issue
Dec 19, 2019
* If the end result is maps.Params, we need to lower the keys * Also make sure to handle nested maps of different type Update #6633
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Starting from version 0.60.0
Params
(page, site, etc.) were changed frommap[string]interface{}
toParams
type. This means that we can not merge them with dictionary produced bydict
function:Maybe it makes sense to update
dict
function to returnParams
type too?The text was updated successfully, but these errors were encountered: