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

Implemented basic layout and functionalities for Recipe Search Results page and Recipe Detail page #15

Merged
merged 7 commits into from
Nov 11, 2020

Conversation

coder-tim
Copy link
Contributor

@coder-tim coder-tim commented Nov 11, 2020

Current functionalities:

  1. Recipe Search Results page currently retrieves all the recipes from our database

    • TODO: After I connect Search Results page with Johnathon's Pantry page, I will adjust it so that we retrieve recipes based on ingredients provided by the user
    • TODO: We also need to implement search filters
  2. Recipe detail page currently has no data

    • TODO: This will involve many more files. I will implement the feature for transferring data and make a separate PR so that we keep each PR at a reasonable size.
  3. The Recipe Search Results page uses the <RecipeCard> component

    • @yanisakham this <RecipeCard> component can be used in your FavouritesList page
  4. I had to make adjustments to the routes so that we could navigate between pages without any bugs related to url path names.

Please Note:

  1. I haven't styled the components yet. I'm going to finish implementing the functionalities first before adding aesthetics to these components._

  2. Some of the components have placeholder text on them

Recipe Search Results Page:
search-results-page

Recipe Detail Page:
recipe-detail

yanisakham
yanisakham previously approved these changes Nov 11, 2020
@@ -0,0 +1 @@
[1110/201713.186:ERROR:directory_reader_win.cc(43)] FindFirstFile: The system cannot find the path specified. (0x3)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this supposed to be here, haha

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yea, it shouldn't

@@ -3,6 +3,7 @@
CONSTANTS.APP_NAME = 'ChefCoPilot';

CONSTANTS.BACKEND_URL = 'https://chefcopilotbackend.herokuapp.com';
CONSTANTS.LOCAL_HOST_BACKEND_URL = "http://localhost:3001";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe for now can we just set CONSTANTS.BACKEND_UR as the local host url instead of making a local_host constant? just comment out the https://chefcopilot one

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yup.

@coder-tim coder-tim merged commit edf9b0c into master Nov 11, 2020
@coder-tim coder-tim deleted the tim-recipe-search-results-page branch November 12, 2020 13:43
@coder-tim coder-tim self-assigned this Nov 15, 2020
@coder-tim coder-tim added the feature Feature implementation label Nov 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants