Skip to content

Chaining with use method#101

Merged
nateps merged 2 commits intoshare:masterfrom
matt-oakes:chaining
Aug 25, 2016
Merged

Chaining with use method#101
nateps merged 2 commits intoshare:masterfrom
matt-oakes:chaining

Conversation

@matt-oakes
Copy link
Copy Markdown
Contributor

A quick PR which adds the missing unit test from #69.

Allows code such as:

backend.use("submit", function(shareRequest, next) {
    console.log("submit");
    next();
  })
  .use("query", function(shareRequest, next) {
    console.log("query");
    next();
  });

droganov and others added 2 commits July 26, 2016 23:47
To allow
backend
    .use( "submit", function( shareRequest, next ){
        console.log( "submit" )
        next()
      }
    )
    .use( "query", function( shareRequest, next ){
        console.log( "query" )
        next()
      }
    )
@coveralls
Copy link
Copy Markdown

coveralls commented Jul 26, 2016

Coverage Status

Coverage increased (+0.002%) to 95.654% when pulling d5ce47d on matto1990:chaining into 5dda246 on share:master.

2 similar comments
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.002%) to 95.654% when pulling d5ce47d on matto1990:chaining into 5dda246 on share:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.002%) to 95.654% when pulling d5ce47d on matto1990:chaining into 5dda246 on share:master.

@nateps
Copy link
Copy Markdown
Contributor

nateps commented Aug 25, 2016

Awesome. Thanks!

@nateps nateps merged commit eefc84b into share:master Aug 25, 2016
@matt-oakes matt-oakes deleted the chaining branch August 25, 2016 19:50
Sign up for free to 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.

4 participants