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

Show items and added event listeners #48

Merged
merged 10 commits into from Jun 20, 2022
Merged

Show items and added event listeners #48

merged 10 commits into from Jun 20, 2022

Conversation

Massacre-Saint
Copy link
Contributor

@Massacre-Saint Massacre-Saint commented Jun 19, 2022

Created function to add the total of selected key/value in the array.
Rendered the items and buttons to DOM.
Connected the create item to form.
Connected the edit item to form.
Connected the checkout button to form.

@Massacre-Saint Massacre-Saint added this to the Part 3: CREATE milestone Jun 19, 2022
@netlify
Copy link

netlify bot commented Jun 19, 2022

Deploy Preview for fluffy-kitten-9d4cc8 ready!

Name Link
🔨 Latest commit c62d762
🔍 Latest deploy log https://app.netlify.com/sites/fluffy-kitten-9d4cc8/deploys/62aea1c2bdd5b600085c6af9
😎 Deploy Preview https://deploy-preview-48--fluffy-kitten-9d4cc8.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Contributor

@gitNitroTitan gitNitroTitan left a comment

Choose a reason for hiding this comment

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

Everything looks great so far. Time for everyone to CRUD on items. I just noticed that on the form, there is a title order name instead of item name on the edit item form. Maybe we can change that on the next push. All is functional for your part though. Good job with using accumulator and the reduce method! Impressive!

Copy link
Contributor

@nishayaraj nishayaraj left a comment

Choose a reason for hiding this comment

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

Looks awesomeJacob.
In the view orders, on cards, the closed status data shows just false. Maybe in the next push we can change it to Closed status: Closed or something similar for better readability.

@Massacre-Saint Massacre-Saint merged commit 89c2b63 into main Jun 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
POS System Team 1
Awaiting triage
Development

Successfully merging this pull request may close these issues.

None yet

4 participants