M8tth3 / m8th3_2.0

Apache License 2.0
0 stars 0 forks source link

CPT #4

Open M8tth3 opened 8 months ago

M8tth3 commented 8 months ago

Link To Blog Link To Video

DavidL0914 commented 8 months ago

1.07/1: Submitted early and have great feature and demo

On the video specifically: Your video overall is solid. It demonstrates your feature nicely and explains some technical parts of how your feature works (the switch to database). Some things that can be improved for CPT include your ending and captions. Your captions could be a lot shorter and concise so it's easier to read and follow, but the placement of them is fine. I would also suggest an ending page for your video to summarize your features as it remains on the screen after the video is playing. I think your video could also be a bit more technical to be CPT ready.

On the blog post specifically: Make sure utterances are on!

I think overall the blog post meets all requirements and explains your features and how they match with CB CPT requirements. However, I think you can expand on certain questions to be more specific and include more technical terms. For example, for your first question I would talk about how your feature utilizes parts of CRUD like create and read, and then explain how they are conditionals based off the user logging in. Just try to utilize more technical terms and explain your features more to go above and beyond for the CPT requirements.

For your collection question, you could go further by explaining how your feature manages complexity and fulfills your program's purpose. Your visual is pretty good.

I like how you put images of your code under your explanations to how your feature matches the CPT requirements. The images are well explained and well marked up to demonstrate what you are saying. I also like how you link your commits next to each big feature, this is very good.