Skip to content
This repository has been archived by the owner on Jul 12, 2019. It is now read-only.

Throwing a better error when create returns a list with one item #44

Merged
merged 1 commit into from
Jul 31, 2017

Conversation

BrunoBernardino
Copy link
Contributor

@BrunoBernardino BrunoBernardino commented Jul 31, 2017

@BrunoBernardino BrunoBernardino self-assigned this Jul 31, 2017
@BrunoBernardino BrunoBernardino requested a review from kola-er July 31, 2017 15:55
Copy link
Contributor

@kola-er kola-er left a comment

Choose a reason for hiding this comment

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

Good to go. Would love to hear your response to my curiosity left in the comment.

@@ -11,7 +11,7 @@ const createIsSingle = {
name: 'createIsSingle',
shouldRun: isCreate,
run: (method, results) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Just curious, what's the purpose of argument method. It seems not used.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It isn't used here, but it's used in other checks, like this one: https://github.com/zapier/zapier-platform-core/blob/master/src/checks/is-search.js#L3

@BrunoBernardino BrunoBernardino merged commit 492b30f into master Jul 31, 2017
@BrunoBernardino BrunoBernardino deleted the fix-reject-list-with-single-item-in-create branch July 31, 2017 17:35
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.

2 participants