getJSON deprecated; replace with resources.Get#70
getJSON deprecated; replace with resources.Get#70whyboris wants to merge 1 commit intoloup-brun:masterfrom
Conversation
|
To clarify: your current code will try to open the |
|
Thank you for the PR. It definitely makes sense. I will try to make time to manage the state of this project in the coming weeks (such as setting up a versioning system in order to better track features/evolutions of the module). |
|
I've now adapted I love how easily-customizable the system you created is 💖 I'm messing with the styling since I don't need APA I just want references to look good (see PR: whyboris/utilitarianism.net#127 -- for more than 3 authors it instead lists first author + "et al." 😎, etc) I'm happy to assist with this PR or your project 😄🤝😊 |
|
@xbc5 could you have a look at this? I tried using it but now Hugo not only produces a warning, but also an error, so you effectively can't generate the site. I'm running v0.132.2. |
|
Sorry, I just realised #62 is actually from 2022, not sure if you still have the time to do it |
|
Yeah, I don't have the time. I currently am working on some other things, and I haven't used Hugo in 1-2 years, so I cannot be confident about what I am merging. loup-brun added me as a maintainer because I intended to take it on, but things have changed for me. I will be creating a Hugo project in the next month or so -- once I familiarise myself with Hugo again, perhaps I will take another look at this. @loup-brun, you can remove me as a maintainer if you think that it's an issue. |
getJSONis deprecated in Hugo 0.123.0 see: https://gohugo.io/functions/data/getjson/Recommended alternative: https://gohugo.io/functions/data/getjson/#global-resource-alternative
Thank you for creating
hugo-cite🙇♂️ I am using it on https://github.com/whyboris/utilitarianism.netI was inlining
{{< cite "Singer1972" >}}over 500+ times across my markdown files and the compilation time was about 10 seconds. After moving toresources.Getthe compilation is back to < 500ms 🚀I hope you can merge in this change ❤️
resources.Getmethod requires thejsonfile to be located in/assets🤔 so we may need modification for this to work.