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

Use the Product Object to contain info from API #8

Merged

Conversation

@HaroldFS
Copy link
Contributor

HaroldFS commented Mar 10, 2020

// TODO: use class Product
// TODO: Change this to be an array of class Product

// TODO: use class Product
// TODO: Change this to be an array of class Product
Copy link
Member

patrick-fs left a comment

LGTM!

@HaroldFS HaroldFS merged commit 1544583 into master Mar 10, 2020
@HaroldFS HaroldFS deleted the harolddavis/ch107799/use-the-product-object-with-api branch Mar 10, 2020
HaroldFS added a commit that referenced this pull request Mar 20, 2020
* Create an API Service

- Create the singleton
- Add main endpoint as a property
- Setup header for method to get shop items

* Allows Local Networking

* Style consistancy.

* - change name to product
- fixed error guard statment

* Changed the Directory name of where the project lives to swift

* Adjusted getSho[[eItem Closure
- Improved Try by adding a `catch` for errors

* Added a config file (Preferences) to hold hardcoded Values
Made baseURL a computed property that retrives the URL from the config file.

* removed test

* removed duplicate files
renamed file from shopeeItem to product

* Fix up Hierarchy so everything is nice an organized

- Make folders to organize files
- set up project for programmatic UI

* Removed un-needed image

* Use the Product Object to contain info from API (#8)

// TODO: use class Product
// TODO: Change this to be an array of class Product

* Allows Local Networking

* rebase from master

* Moved Scene and app delegate out of Resource folder...

* cruft

* removing more duplicate code

* removing duplicates

* removing duplicate APIService
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.