Tyler929 / WalkieTalkies

1 stars 2 forks source link

Review Ticket Week 3 #56

Open katiehickman opened 2 years ago

katiehickman commented 2 years ago

Tyler: GitHub admin tangibles: Ticket for Issue Commit, pull request procedure: Management Plan Branching procedure :Management Plan Individual Section: Here

Evidence of Pull Requests: Kamya + Tyler + Kamya Again

Commits for each individual: Tyler + Katie + Kaavya + Kamya

Issues completed by each individual: Tyler + Kaavya + Kamya + Katie

AP MCQ Test and Corrections <link to results>

Overall Grade: 7.75 / 8, we don't have pull requests from each individual (only 3 total pull requests) but repository shows evidence of pull request usage and includes both branching and pull request procedures within the management plan. Also, The history of pull requests suggests effort towards the implementation of pull requests in the future, although this may be discussed further with the scrum team.

rkaavya commented 2 years ago

Deployment manager tangibles Update procedure: https://github.com/Tyler929/WalkieTalkies/wiki/Management-Plan#raspberry-pi-generation-3-deployment-guide Nginx config:https://github.com/Tyler929/WalkieTalkies/wiki/Management-Plan#raspberry-pi-generation-3-deployment-guide Service config:https://github.com/Tyler929/WalkieTalkies/wiki/Management-Plan#raspberry-pi-generation-3-deployment-guide

Securing Process: https://cs-p-hq.slack.com/files/U02BNQBFNNP/F02QP8QU5UL/image_from_ios.mov

Site: https://walkietalkies.cf

AP MCQ Test and Corrections: https://docs.google.com/document/d/1vo8cymb8qllolHUWJVqj0A-CIg5Xe_fPEnlQ07DNSUk/edit?usp=sharing

Overall Grade 7.5/8, I wasn't able to pull the latest version of the website due to a conflicting merge error, but I was able to pull some of the code.

kamyamah commented 2 years ago

Design Manager tangibles Layout. https://github.com/Tyler929/WalkieTalkies/blob/main/templates/index.html Front End Feature Screen http://127.0.0.1:5000 Front End Screen 2: http://127.0.0.1:5000/artists/ Front End Screen 3: http://127.0.0.1:5000/famous/ https://github.com/Tyler929/WalkieTalkies/commit/7531004fa7a168a2573f9f5557bff71f4b6db1a1 AP MCQ and corrections Overall Grade: 7.75 Because I completed the tasks that I had to do additional justifications are in the read me. Also I was the only one to get the pull request to show up on the git hub so I thought that was a notable event I created a new branch for my commits specifically and I was the only one out of my group who filled the requirement. Although during the live review I did have the body tags I deleted them after the fact and the code is completely functioning just goes to show that I did not work last minute and I had all my work done despite a power outage and limited responses from instructor via slack. I also completed my quiz and my corrections were thoroughly done in a timely manner. The website was really pretty and all the runtime features like the artist spotlight, famous artists, and the home page were completed by me I had no additional help on those features. Despite the small Hiccups in the live review I was able to delete the body tags and put in extra effort beyond class to make sure and beautify the website. I would also like my points back from three weeks ago week 1 I was promised my points for linking my create task notes but never received the credit for my work.

katiehickman commented 2 years ago

Scrum Master and Technical Officer PBL Highlights. readme time box link Video link to one minute video

Scrum Master tangibles Scrum Board GitHub Scrum Board <tickets in review link> <in progress link> Scrum Board <post winter break backlog link>

Technical officer tangibles Database with CRUD: Runtime <link to CRUD page runtime> View (frontend html) <link to code supporting crud routes and logic> Model (Backend Definition - defining table class and functions for CRUD operations) <link to class and crud> Control (backend form request and API - routes between model and view) <link to control>

AP MCQ Test and Corrections <link to results>

Katie overall grade: 5.9/6 Team, 2/2 Individual: 61/67 Score on MC Reasoning: I did all of my tasks for both the scrum master and technical officer. I was really proud to get the CRUD database and search function working on our website. I also customized it to fit our website's theme. I was also proud of the 1 minute video I made for the post-winter-break plans, current PBL highlights, and asynchronous database access. I would take .1 off of our team grade because some of our layout pages have too many "body" areas. We are in the process of fixing that.

We all fixed our Week 1 create task plan because we got 1 point removed from that week. If possible, we would love the 1 point back for that week. Here is the link to the create task plans on our wiki and a screenshot of them in our notes on December 6th.

Screen Shot 2021-12-17 at 10 20 05 AM