Skip to content

Conversation

@EAguayodev
Copy link
Contributor

Requested approval for removing margin and width on the search engine due to it going off screen

cmtakara and others added 30 commits February 28, 2023 11:39
ct - testing branch and merge
feat initial create react app ran TBD-1
TBD-6 create empty express applictation
* updated controllers/recipe to support search

* updated controllers/recipe to support search

* took out comments

* removed commented out consoles and rebased
* TBD-13 first push for create search parent

* adding input html and css from codepen

* TBD-13 updating search.js page with new code

* adding search.css code

* code changes to search.js

* removing import '/.index.css'

* updating import to read import 'search.css'

* changing searchInput to searchPage

* TBD-13 updates for rebase

* original rebase

* adjusting back to original code for app.js

* ; to import

* changning import bug searchpage

* deleting search.js

* TBD-13 first push for create search parent

* adding input html and css from codepen

* TBD-13 updating search.js page with new code

* adding search.css code

* code changes to search.js

* removing import '/.index.css'

* updating import to read import 'search.css'

* changing searchInput to searchPage

* TBD-13 updates for rebase

* original rebase

* adjusting back to original code for app.js

* ; to import

* changning import bug searchpage

* deleting search.js

* fixing search page

* fixed import search issue
* updated controllers/recipe to support search

* updated controllers/recipe to support search

* took out comments

* removed commented out consoles and rebased

* updated search strategy to find all ingredients

* removed console and ran linter
cmtakara and others added 7 commits April 3, 2023 11:59
* updates for recipe page, including helmet for cors

* updated useEffect dependencies

* commented out setting id until it can be used

* commented out useparams until it can be used
* Added Get Recipe by ignored

* Linting. Updating url to proper root. Using findById explicitly.

---------

Co-authored-by: Ramraj <ramraj.guptha@gmail.com>
* Adding common axios api file

* Adding axios api config

* Adding link to results

* rebase

Force merge since we are out of time.
* Adding common axios api file

* Adding axios api config

* Adding link to results

* rebase

* trying cors settings
@BboyAkers BboyAkers merged commit b9ba120 into dallassoftwaredevelopers:main Apr 6, 2023
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.

5 participants