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

Add trimmed User #37

Merged
merged 4 commits into from
Jul 12, 2019
Merged

Add trimmed User #37

merged 4 commits into from
Jul 12, 2019

Conversation

takanakahiko
Copy link
Contributor

fix #25

designd by #25 (comment)

abraham
abraham suggested changes Jul 11, 2019
View reviewed changes
Copy link
Owner

@abraham abraham left a comment

Choose a reason for hiding this comment

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

I think FullUser and TrimmedUser should be exported so that functions can specify if they require a FullUser.

types/user.d.ts Outdated Show resolved Hide resolved
types/user.d.ts Outdated Show resolved Hide resolved
@takanakahiko
Copy link
Contributor Author

Thanks !

@takanakahiko
Copy link
Contributor Author

Fixed. Please review.

@abraham abraham merged commit b331ca7 into abraham:master Jul 12, 2019
@abraham
Copy link
Owner

abraham commented Jul 12, 2019

Awesome! Thanks!

@abraham
Copy link
Owner

abraham commented Jul 12, 2019

v0.3.0 has been released.

@AaronBuxbaum AaronBuxbaum mentioned this pull request Jul 28, 2019
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@abraham abraham abraham approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

User in Status may be trimmed
2 participants
@takanakahiko @abraham