new commit - #18
Conversation
updated
Drbaltar
left a comment
There was a problem hiding this comment.
Hey Lansah,
Your JSON was missing a ']' at the end of the file. Also, I see that the filter functionality is already being done by the front-end that Lance set up so there is no need to pass in a search keyword. Finally, I would recommend combining the details and tv/movie overview into one file. See below for an example of what I am referring to:
{
"tv_shows":[
{
"tvShow_ID":0,
"isTV":true,
"rating":false,
"isWatched":false,
"originalTitle":"TV show Title",
"releaseDate":"2019-02-22",
"overview":"This TV show is about love",
"poster_path":null,
"trailer_path":null,
"vote_average":0,
"vote_count":0
}
],
"movies":[
{
"movieID":0,
"isTV":false,
"rating":false,
"isWatched":false,
"originalTitle":"Movie Title",
"releaseDate":"2019-02-22",
"overview":"This movie is about crime",
"poster_path":null,
"trailer_path":null,
"vote_average":0,
"vote_count":0
}
]
}
|
Lance, are you talking about labeling the 'movieID' and 'tvShowID' to just 'id' for both? |
Update Movie and TV ID key.
No description provided.