Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Natyka C Tuner App Completed #12

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

NatykaC
Copy link

@NatykaC NatykaC commented Nov 10, 2023

How did this assignment go?

Completion 1 -5

My self-assessed completion score is 4.5

5: I did everything required and attempted at least one bonus

4: I attempted all parts of the lab and completed most/all of the required parts of the assignment

3: I was able to attempt most parts of the lab and got some of the required parts done

2: I was only able to make a little progress

1: I got completely stuck and/or did not have enough time to work on this assignment

Comfort 1- 5

My self-assessed comfort score is 4

5: This assignment was a breeze! I want more challenges

4: This assignment was a bit challenging but I learned a lot

3: There were some required parts of this assignment that were too hard for me to complete

2: I really struggled to understand what to do for this assignment, but I think I still made some progress

1: I feel completely lost and I was not able to further my understanding

Wins

Please list any wins: I finally finished setting up the front end and everything works!!

Struggles

Please list any specific struggles: I created a React App in the front end when I didn't need to.

Other comments

Please add any other things you'd like us to know about this assignment My front end components are in the frontend/tuner app

quicktest

… and artist are required. Testing both routes with Postman and they are working correctly.
…o the put route. Updated the error handling messages for all routes. Tested in Postman and all routes are working correctly.
… routes are completed, connected, and functional.
…ed albums with the table called songs inside. Created a controller file and a queries file for the albums table. Also updated the songs controller and queries file to account for make it possible to do crud with the merged param url of albums/:id/songs. Changed the validations to checkAlbum instead of check song. Tested out the updated routes with postman and all the routes are working correctly. Still need to update the front end to get the albums and songs to show on the tuner app when the full url is entered.
…jsx, and AlbumDetails.jsx to handle the album functions. Changed the fetch from songs to albums and I am able to display a list of all albums. Still unable to see the songs for each album.
…ble to get all the albums and each indivdiual album. I still cannont get the songs for each album. All routes with the merged params are working on postman.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant