Skip to content

Warn about injectOptionsFromRemoteContext - #3080

Merged
bajtos merged 1 commit into
masterfrom
feature/warn-on-injectOptionsFromRemoteContext
Jan 10, 2017
Merged

Warn about injectOptionsFromRemoteContext#3080
bajtos merged 1 commit into
masterfrom
feature/warn-on-injectOptionsFromRemoteContext

Conversation

@bajtos

Copy link
Copy Markdown
Member

The option injectOptionsFromRemoteContext was added to LoopBack 2.x only and is not available in LoopBack 3.x (and newer).

When a model with this option is encountered, we are printing a warning message now, to let the user know about this change between 2.x and 3.x.

Connect to #1495, see also #3048 which implemented the flag in 2.x

@superkhau PTAL
@crandmck could you please review the message?

The option injectOptionsFromRemoteContext was added to LoopBack 2.x only
and is not available in LoopBack 3.x (and newer).
When a model with this option is encountered, we are printing a warning
message now, to let the user know about this change between 2.x and 3.x.
@bajtos

Copy link
Copy Markdown
MemberAuthor

The current message:

TestModel is using model setting injectOptionsFromRemoteContext which is no longer available.
Please rework your app to use the offical solution for injecting "options" argument from request context,
see http://loopback.io/doc/en/lb3/Using-current-context.html

@cgolecgole removed the #review label Jan 10, 2017
@bajtos
bajtos merged commit b86eac5 into masterJan 10, 2017
@bajtos
bajtos deleted the feature/warn-on-injectOptionsFromRemoteContext branch January 10, 2017 14:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@bajtos@superkhau@cgole