Skip to content

Conversation

@cbush
Copy link
Contributor

@cbush cbush commented Jan 28, 2019

@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 76.739% when pulling 0c299d2 on cbush:docsp-3944-readme into d9ab6ab on mongodb:master.

Copy link

@MongoCaleb MongoCaleb left a comment

Choose a reason for hiding this comment

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

A couple of small notes that I put in the js sdk but apply to the other, as well.

})
```

Assuming you've configured your Stitch application to have a function named "echoArg" that returns its argument, you should see a message like:

Choose a reason for hiding this comment

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

Is it worth showing that function here, too?

This library depends on [js-bson](https://www.npmjs.com/package/js-bson).
The Stitch SDK depends on [bson](https://www.npmjs.com/package/bson).

As a convenience, the SDK includes the `BSON` library, and you can import it as you would import other classes and values from the SDK.

Choose a reason for hiding this comment

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

So we say the Stitch SDK depends on it, and then we say the SDK includes it...

- Adds more links to other docs in the three READMEs
- Attempt to clarify and streamline some of the content
@cbush cbush force-pushed the docsp-3944-readme branch from 0c299d2 to 8820a43 Compare January 29, 2019 17:45
@cbush cbush merged commit 75126d8 into mongodb:master Jan 29, 2019
@cbush cbush deleted the docsp-3944-readme branch January 29, 2019 18:36
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