Info
Source on GitHub
Smart Contract Address
0:97344bd09ee1b3b84c2baee9f1c11df108928f2dc864937f88c50dcb015c66b9
Source on GitHub
Smart Contract Address
0:54f31b12814165f64b564e51d351e4e04ac0a990a764c3ac0315221cbdda8f82
Testing and Issues
You can test this entry and submit issues during the testing period of the Blockchain Contest, Stage 2: Bonus Prize contest.
Entries with serious issues will not be able to win the contest, but even minor issues might be important for overall results.
Voting
Comments
Hi,
website for my contract is available here:
https://5eraph.github.io/ticketeer/
And video here: https://github.com/5eraph/ticketeer/blob/master/ticketeer.mp4
Video is as well available on the website after landing section.
All issues has been fixed fully or to the extend possible with included additional information on how to solve (e.g. issue regarding users is client side one and as such has to be implemented properly on the client side as described: https://github.com/5eraph/ts/blob/master/Verification%20Strategies.md)
website for my contract is available here:
https://5eraph.github.io/ticketeer/
And video here: https://github.com/5eraph/ticketeer/blob/master/ticketeer.mp4
Video is as well available on the website after landing section.
All issues has been fixed fully or to the extend possible with included additional information on how to solve (e.g. issue regarding users is client side one and as such has to be implemented properly on the client side as described: https://github.com/5eraph/ts/blob/master/Verification%20Strategies.md)
I noticed that the web page has some issues with container overflows on the phones (responsive UI related), so I just added the fix. I acknowledge that it may or may not be included in final review based on judge decision. I made it for phone users and their better experience. In case you want me to switch it back to review form before fixes just leave me comment here and I will reverse it. I can technically deploy in parallel as well.
NOTE: desktop or bigger screen experience wasn't affected with the fix.
NOTE: desktop or bigger screen experience wasn't affected with the fix.
You have not added any comments yet...
by rating
Issues
WEB SCORE: 2.33
Opening Section – 2
Concept Explained – 3
Language – 2
The opening lacks basic info: how does this work? Who is it for? Why do I need it? Moves on to a rather vague video, then to an explanation (that could be less technical and could use some illustrations/diagrams) with grammar mistakes and typos. All texts could benefit from a thorough proofreading by a native speaker.
VIDEO SCORE: 2.83
Narration – 3
Visualization –3
Concept Explained – 1
Bonus: Language +0.25
Bonus: Soundtrack +0.25
The narrative is vague and doesn't explain why the smart-contract is special except for being "distributed and integrated in platforms of your choice". It would have been better to focus on the particular benefits of the smart contract for the particular use case – instead of using general phrases like "Protect your customers, streamline your business" and "All is included. All features you might ever need".
=====
Base Reward: EUR 3,500
OLD BUGS FIXED: 4/5
NEW CRITICAL ISSUES:
“add available_tickets” saves received count instead of _newAvailableTickets. refund_ticket doesn't return back paid, buy_ticket don‘t always return back correct amount. Instead of providing verification strategies contract could require the contract’s address to be appended to verification cell before signing it.
NEW MINOR ISSUES:
16-bit seqno has a big chance to overflow, opening possibility for replay attacks in the future.
Bug Penalty: EUR 1,000
======
Total: EUR 2,500
Opening Section – 2
Concept Explained – 3
Language – 2
The opening lacks basic info: how does this work? Who is it for? Why do I need it? Moves on to a rather vague video, then to an explanation (that could be less technical and could use some illustrations/diagrams) with grammar mistakes and typos. All texts could benefit from a thorough proofreading by a native speaker.
VIDEO SCORE: 2.83
Narration – 3
Visualization –3
Concept Explained – 1
Bonus: Language +0.25
Bonus: Soundtrack +0.25
The narrative is vague and doesn't explain why the smart-contract is special except for being "distributed and integrated in platforms of your choice". It would have been better to focus on the particular benefits of the smart contract for the particular use case – instead of using general phrases like "Protect your customers, streamline your business" and "All is included. All features you might ever need".
=====
Base Reward: EUR 3,500
OLD BUGS FIXED: 4/5
NEW CRITICAL ISSUES:
“add available_tickets” saves received count instead of _newAvailableTickets. refund_ticket doesn't return back paid, buy_ticket don‘t always return back correct amount. Instead of providing verification strategies contract could require the contract’s address to be appended to verification cell before signing it.
NEW MINOR ISSUES:
16-bit seqno has a big chance to overflow, opening possibility for replay attacks in the future.
Bug Penalty: EUR 1,000
======
Total: EUR 2,500
Nobody added any issues yet...