search

mean-expert-official / loopback-sdk-builder

Tool for auto-generating Software Development Kits (SDKs) for LoopBack
Other
399 stars 175 forks source link

(2.1 Beta) SDK tries to export real.time module when generating with real time disabled #140

Closed bostondv closed 8 years ago

bostondv commented 8 years ago

Run ./node_modules/.bin/lb-sdk server/server path/to/client/src/app/shared/sdk -l angular2 -d ng4web -i disabled

Generated SDK includes an export for app/shared/sdk/services/core/real.time.ts within app/shared/sdk/services/core/index.ts but real.time.ts doesn't exist.

With real time module disabled, this export shouldn't exist I think :)

jonathan-casarrubias commented 8 years ago

@bostondv yup that is right, looks like there is a missing validation... That is why the realtime does not exist, but the reference shouldn't neither..

I will fix this issue ASAP.

BTW: In the meanwhile, If you don't need the realtime functionality, then you may want to try the latest stable version which is the 2.0.10. That version does not include anything related to the RealTime service and FireLoop.

Cheers Jon

bostondv commented 8 years ago

Awesome, I don't need it right now so I'll try 2.0.10 :)

On Tue, Oct 4, 2016 at 11:06 AM Jonathan Casarrubias < notifications@github.com> wrote:

@bostondv https://github.com/bostondv yup that is right, looks like there is a missing validation... That is why the realtime does not exist, but the reference shouldn't neither..

I will fix this issue ASAP.

BTW: If you don't need the realtime functionality, then you may want to try the latest stable version which is the 2.0.10. That version does not include anything related to the RealTime service and FireLoop.

Cheers Jon

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mean-expert-official/loopback-sdk-builder/issues/140#issuecomment-251416170, or mute the thread https://github.com/notifications/unsubscribe-auth/AAIKtj2hMjAjsWW8oTQ4X1J3pgvjxTiEks5qwmt7gaJpZM4KNtoy .

jonathan-casarrubias commented 8 years ago

Fixed