Hi @bostrot, nice project though I have not checked the app.

A few tips for your next contributions:

• @utopian-io recommends you make small multiple commits which makes it easier for a moderator to check and understand your code easily. Your most recent and important commit is a very large one.

• Your most other commits are more than 14 days old though they are planning to increase the commit max age to 28 days but you can use Pull Request and Merge to get over this. Here is a reference.

A good project nonetheless. :)

Hey @reazuliqbal
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!

Hi, thank you for the information. I didn't expect that and will (hopefully) do better in future :)

Thanks for the contribution, @bostrot. As @reazuliqbal has already mentioned some things to keep in mind here and I have already left a comment here I won't go over the same things again in this comment.

Instead I will focus on reviewing this commit, which is what I'll base your score off of. For future reference it would've been better if the development contribution covered and explained the feature(s) added in the above commit instead of the entire project (for project introductions you can use the blog category instead).

Anyway, let's have a look shall we?

• I'm not familiar with Dart and since there are no comments it's not really clear to me where the magic numbers in this come from. Wouldn't it be better to use constants declared somewhere else for this?
• The amount of characters per line is pretty high in some places. Not sure if there aren't some best practices to keep the line width below a certain number, but I think it would definitely improve the readability of the code (had to scroll horizontally in some places).
• There aren't really that many comments (function declarations would be nice at least) and there is quite a lot of commented out code left in there. In my opinion it would be better to remove those unused snippets, but if you might need them in the future then I understand.

Other than that everything seems to look fine to me and I don't have any other remarks - great work! I will once again (it's that good) recommend you read this amazing guide on how to achieve a higher score in the development category before creating another contribution, as it will definitely help you receive a higher reward (it's exponential and not linear).

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]

Well thank you again for this detailed anwer. I will add more comment in the future. The magic numbers you mentioned is a function that returns an Array (in Dart: list) of possible video qualities. This happens with checking whether the desired quality exists in json_metadata.
Wrapping lines can be done, though Dart especially Flutter is known for those heavy indents. But its true I could have avoided a few of these.
Some of the unused parts are important and some not. Will sort this out.

Thank you for your review, @amosbastian!

So far this week you've reviewed 3 contributions. Keep up the good work!

Hi @bostrot! We are @steem-ua, a new Steem dApp, computing UserAuthority for all accounts on Steem. We are currently in test modus upvoting quality Utopian-io contributions! Nice work!

Hey @bostrot
Thanks for contributing on Utopian.
We’re already looking forward to your next contribution!

Want to chat? Join us on Discord https://discord.gg/h52nFrV.

Vote for Utopian Witness!

I can’t get that app to work at all!

Posted using Partiko iOS