search

fullcube / loopback-ds-readonly-mixin

A mixin to enable readonly properties for loopback Models
MIT License
51 stars 19 forks source link

mixinSource docs are wrong/misleading? #4

Closed doublemarked closed 7 years ago

doublemarked commented 8 years ago

Hey - we upgraded to loopback-boot@2.8.2 and changed our loopback-ds-readonly-mixin config to avoid the deprecation warning. After doing so, the mixin stopped working. I believe your docs are wrong - the mixin source needs to reference the lib subdir, like this:

    "mixins": [
      "loopback/common/mixins",
      "../node_modules/loopback-ds-readonly-mixin/lib",
      "../common/mixins"
    ]

Once we changed that the mixin started loading again.

geekguy commented 8 years ago

@doublemarked : You are right. lib subdir is required.

faustomorales commented 8 years ago

Got me up and running! Thanks.

ebarault commented 7 years ago

certainly worth updating the Readme?

mrfelton commented 7 years ago

Have updated the docs. Thanks.