Add search - Githubissues

pan-alex commented 2 years ago

Description of the PR:

Added search feature on the /flix route. Users enter a movie title into the input and hit submit, and the page displays key information about the movie.
The search by movie title is case insensitive. It first searches for exact matches. If no exact match is found, it will search by partial match.
The movie data comes from the sample_mflix sample dataset from MongoAtlas. The MongoAtlas account needs to be configured to load in these datasets before merging this PR. If reviewing this code, message me so that I can provide a DB string for an already-configured database.

Changes in this PR include:

Add search schema to interact with the MongoDB (sample_mflix database)
Add search routes & controller to receive user input, and respond with a json object containing movie data.
Update views/flix.ejs and public/main.js to make the request and display the data in the DOM once loaded.
Added very minimal CSS styling to replicate the rough layout of the mock-up.

Co-authored by @edward2550

pan-alex commented 2 years ago

This is to address issue #11

floorrinse commented 2 years ago

I noticed some changes you made that would overwrite what's currently in the main branch. Github also is letting you know you'll have to resolve those conflicts before you merge this pr (fyi i pushed up a profile nav menu change that would be overwritten)

pan-alex commented 2 years ago

From what I can tell this is just the style.css? I know that several branches have made changes to the CSS; I purposely only added 3 declarations so it should be simple to resolve (by accepting both changes) at merge.

floorrinse commented 2 years ago

From what I can tell this is just the style.css? I know that several branches have made changes to the CSS; I purposely only added 3 declarations so it should be simple to resolve (by accepting both changes) at merge.

There's also the views/ejs.file where I added in the menu component if you check out the main branch code: https://github.com/CadmiumRed40/FindAFlick/blob/main/views/flix.ejs#L11

I can add it back in later though after this is merged

pan-alex commented 2 years ago

Resolved conflicts and merged with main in 4877700
Re-configured MongoDB with @TonyHerbert22ATX.

Merging PR.

CadmiumRed40 / FindAFlick

Add search #24