-
Notifications
You must be signed in to change notification settings - Fork 312
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CU-864dmfjy9 - Upgrade Ghost Market API to v2 #2456
Conversation
Task linked: CU-864dmfjy9 Upgrade Ghost Market API to v2 |
app/actions/nftGalleryActions.js
Outdated
) | ||
|
||
// eslint-disable-next-line no-console | ||
console.log(results) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably remove this log
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I forgot to remove this log. Solved
app/actions/nftGalleryActions.js
Outdated
|
||
return { results: previousResults.concat(results), page, count } | ||
} | ||
if (!assets || !assets.length || !count) throw new Error('No assets found') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think keeping the previous pattern of returning an empty payload makes more sense than returning an error here as there is no error that occurred - no NFT's is not really an error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok. Solved
f5ef53f
to
87b75ec
Compare
What current issue(s) from Trello/Github does this address?
What problem does this PR solve?
How did you solve this problem?
How did you make sure your solution works?
Are there any special changes in the code that we should be aware of?
Is there anything else we should know?