Skip to content
This repository was archived by the owner on Oct 11, 2022. It is now read-only.

Comments

replace the invocation with the real one#118

Open
cxdongjack wants to merge 1 commit intostubailo:develfrom
cxdongjack:devel
Open

replace the invocation with the real one#118
cxdongjack wants to merge 1 commit intostubailo:develfrom
cxdongjack:devel

Conversation

@cxdongjack
Copy link

  • to support the Meteor.userId() in the method

- to support the Meteor.userId() in the method
@dotansimha
Copy link

👍


var handlerArgs = options.getArgsFromRequest(req);
var handlerReturn = handler.apply(methodInvocation, handlerArgs);
connection: {},
Copy link

@dotansimha dotansimha May 10, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can use the req as the connection here, so the Meteor methods will have access for the HTTP request object.

@dpatte
Copy link

dpatte commented May 22, 2017

this package resolved all my Simple:Rest issues

@rijk
Copy link

rijk commented Sep 25, 2018

To get this to work, I also had to add 'ddp-common' in the package.js file:

  api.use([
    'check',
    'ddp',
    'ddp-common', // <--
    'ejson',
    'meteor',
    'mongo',
    'simple:json-routes@2.1.0',
    'underscore',
    'webapp',
  ], 'server');

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants