RE: SteemChain - Charts & Analytics V1.2.0
Thank you for your contribution. I have tried in the chrome browser, and it shows some errors - the graph shows but the table doesn't.
I read your answers to @amosbastian which also mine. I can also see you have put all the classes in one file operations.py which may not be ideal - you might want to store the sub-classes separately. You might also want to unit test the operations based on historic block data.
Maybe you can disregard the blocks that were more than 2-year old - to reduce the burden of your server.
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? Chat with us on Discord.
Thanks for feedback, what would be advantage to storing the sub classes separately?
The errors are due to the syncing with the blockchain. For the stats displayed it is trying to get data from the last 24H and 30 Days. Since the application is still catching up there is no data for now.
It will make your code base more organised (clearer structure) and reduce the conflicts if many developers are working on the project...
Thanks
Thank you for your review, @justyy! Keep up the good work!