Skip to content

Tried to incorporate recommended changes.#1

Open
hafgan wants to merge 8 commits into
mikeshhoffman:masterfrom
hafgan:master
Open

Tried to incorporate recommended changes.#1
hafgan wants to merge 8 commits into
mikeshhoffman:masterfrom
hafgan:master

Conversation

@hafgan
Copy link
Copy Markdown

@hafgan hafgan commented Jun 1, 2017

Tried to incorporate recommended changes. Also tried to incorporate code from branch. Both ways am getting NaN in order page.

jefflembeck and others added 8 commits April 24, 2017 17:34
The database for pastries had them as an array. This, unfortunately, is
not sustainable for growing inventory in the future if we'd like to
avoid duplicates, etc.
…play what is ordered on a orders page. On index.js - state does not seem to persist when visitng /orders and on OrderList.js - it seems to be returning and being treated as text instead of as HTML. Cliff
…N for all the items. So I tried to incorporate your code from the branched version. Still am getting NaN, Cliff
@jefflembeck
Copy link
Copy Markdown

@hafgan Hey Cliff, it looks like you PR'd this to Mike's repo instead of the main one.

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.

2 participants