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.

Sign up for GitHub

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

Jump to bottom

Update relations generate #18

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Update relations generate #18

wants to merge 5 commits into from

Conversation

ht22pt
Copy link

@ht22pt ht22pt commented Mar 23, 2017

Description

Support relation generation

Related issues

  • Type Object is too hard for access from iOS, it will be one NSDictionary.
  • Not support Array type.
  • Remote method only support basic type (not object, array or ...)
  • Only work with Lb2

Checklist

  • Change to ES6 code
  • Add support generate code with all relations models
  • Support generate with type is Object.
  • Add generate header LoopbackModelImport, on Xcode only need import this file for use all models.
  • Tested working with my project in Xcode 8.3 with Swift3

Thanh Nguyen and others added 5 commits March 23, 2017 09:43
Update format js of ES6
946479e
- Add support include all models have relation with main models.
63e329b 
- Support generate all relation models
- Support all models include with one file it support on Xcode easy for include to build
- Change to ES6 code.
- Add support type object --> will change to NSDictionary.
  + Hard for get from iOS client but can be get. Must find better solution here.
- Test using in Xcode 8 with swift3 ok.
Add import all models need for relations of this model
cc42350
Add LoopbackModelImport all for easy import in Xcode
aff5b14
@ht22pt
Merge branch 'master' into master
e24b11f
@toniopelo
Copy link

This PR sounds good, any plans on merging this soon ?
Thx guys!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ht22pt @toniopelo