Skip to content

fix(getObject): null in return type for items #1183

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

Merged
merged 1 commit into from
Aug 18, 2020

Conversation

dispix
Copy link

@dispix dispix commented Aug 18, 2020

Using getObjects returns an array of records but also null when the object id doesn't match any record. Updating the type to reflect that.

See example of a response where the first object id matches a record and the second (object id "z") doesn't:
image

@dispix dispix added bug TypeScript Problems related to TypeScript definitions labels Aug 18, 2020
@dispix dispix self-assigned this Aug 18, 2020
Haroenv
Haroenv previously approved these changes Aug 18, 2020
@Haroenv Haroenv changed the title Nullable getObjects items fix(getObject): null in return type for items Aug 18, 2020
@Haroenv
Copy link
Contributor

Haroenv commented Aug 18, 2020

seems like it's circle which is partially down, not the tests themselves per se. Can you do a force push?

@dispix dispix force-pushed the fix/get-objects-nullable branch from b2cb768 to 52fab4d Compare August 18, 2020 15:26
@Haroenv Haroenv merged commit cd22566 into master Aug 18, 2020
@Haroenv Haroenv deleted the fix/get-objects-nullable branch August 18, 2020 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug TypeScript Problems related to TypeScript definitions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants