RE: EduSteem - IPFS support (Backend)
Hi again,
Another use of IPFS, that's a good call for hosting images ;)
Let's go for the review:
I'm a bit disappointed, only one feature in this contribution
You should extract your create table queries and create a separate initialization script
https://github.com/bflanagin/EduSteem/commit/6ddcf57b31f4755ca68329979ebb13268fd162e5#diff-59dd6010e6e8047ad9ff08ff243062faR7 => localhost is hard coded it's not really good
Best practice is to use
letothervar
It's really great that you're using IPFS but you should have added a bit more stuff before submitting a contribution in my opinion.
Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.
To view those questions and the relevant answers related to your post, click here.
Need help? Write a ticket on https://support.utopian.io/.
Chat with us on Discord.
[utopian-moderator]
lol, just can't win can I? I put in 3 or 4 features and get the same amount of payout as I get when when I put in 1. Sometimes its due to merit, but most of the time it seems to be on a per moderator basis. I'm planning on writing the "frontend" post today.
I agree about the localhost not needing to be hard coded I'm working on a settings page where people would be able to set those options. I am under the impression that you can ensure the media is locally cached by calling the local ipfs service to fetch the content. I could be wrong about this, so I'll do more reading.
Hey @gregory.latinier
Here's a tip for your valuable feedback! @Utopian-io loves and incentivises informative comments.
Contributing on Utopian
Learn how to contribute on our website.
Want to chat? Join us on Discord https://discord.gg/h52nFrV.
Vote for Utopian Witness!