Skip to content

Update persisted-model.js - #1910

Merged
crandmck merged 1 commit into
masterfrom
fix-findOrCreate-doc
Dec 23, 2015
Merged

Update persisted-model.js#1910
crandmck merged 1 commit into
masterfrom
fix-findOrCreate-doc

Conversation

@crandmck

Copy link
Copy Markdown
Contributor

Correct doc for findOrCreate per loopbackio/loopback-datasource-juggler#766

@crandmck

Copy link
Copy Markdown
Contributor Author

@superkhau Please review and land if it looks OK.

Comment thread lib/persisted-model.js Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

add two spaces in front

@superkhau

Copy link
Copy Markdown
Contributor

Just missing tabs in front of the start and end of comment block. Otherwise, LGTM.

Comment thread lib/persisted-model.js Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

data provided in the second argument?

@crandmck

Copy link
Copy Markdown
Contributor Author

Fixed, ty.

@crandmck

Copy link
Copy Markdown
Contributor Author

OK, fixed the notional where clause.

Comment thread lib/persisted-model.js Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

it is

- Correct doc for findOrCreate.
- Fix indentation
- Remove quotes in example `where`
- Fix typo
- Fix spacing in notional where clause
@superkhau
superkhau force-pushed the fix-findOrCreate-doc branch from 05f3476 to 1af0361 Compare December 23, 2015 01:08
crandmck pushed a commit that referenced this pull request Dec 23, 2015
@crandmck
crandmck merged commit d4040dc into master Dec 23, 2015
@crandmck crandmck removed the #review label Dec 23, 2015
@crandmck

Copy link
Copy Markdown
Contributor Author

Thanks @superkhau !!!

@crandmck
crandmck deleted the fix-findOrCreate-doc branch December 23, 2015 01:11
crandmck pushed a commit that referenced this pull request Jan 9, 2016
bajtos added a commit that referenced this pull request Feb 19, 2016
 * Remove sl-blip from dependency (Candy)
 * Fix race condition in replication tests (Miroslav Bajtoš)
 * test: remove errant console.log from test (Ryan Graham)
 * Promisify Model Change (Jue Hou)
 * Fix race condition in error handler test (Miroslav Bajtoš)
 * Travis: drop iojs, add v4.x and v5.x (Miroslav Bajtoš)
 * Correct JSDoc findOrCreate() callback in PersistedModel (Miroslav Bajtoš)
 * Hide verificationToken (Miroslav Bajtoš)
 * test: use ephemeral port for e2e server (Ryan Graham)
 * test: fail on error instead of crash (Ryan Graham)
 * ensure app is booted before integration tests (Ryan Graham)
 * Checkpoint speedup (Amir Jafarian)
 * Pull in API doc fix from PR into master #1910 (crandmck)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants