Skip to content
This repository has been archived by the owner on Jun 29, 2021. It is now read-only.

allow refType to be defined in @prop (issue #146) #148

Closed
wants to merge 3 commits into from
Closed

allow refType to be defined in @prop (issue #146) #148

wants to merge 3 commits into from

Conversation

mattjennings
Copy link

No description provided.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.8%) to 86.217% when pulling bd43ae7 on mattjennings:master into 7f9f876 on szokodiakos:master.

@coveralls
Copy link

coveralls commented May 12, 2018

Coverage Status

Coverage decreased (-0.8%) to 86.217% when pulling a386f3d on mattjennings:master into 7f9f876 on szokodiakos:master.

@@ -23,6 +23,7 @@ export interface BasePropOptions {

export interface PropOptions extends BasePropOptions {
ref?: any;
refType?: any;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is ˋanyˋ really correct here? What about ˋNumber | String | Bufferˋ?

Copy link
Author

Choose a reason for hiding this comment

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

No, you're right. It uses a mongoose.Schema.Types object, but I'll find the correct way to type it.

@mattjennings
Copy link
Author

I realized I had other commits from my fork coming through on this by accident, I'll create a proper pull request soon.

@Ben305
Copy link
Collaborator

Ben305 commented May 12, 2018

Great, I love clean, focused PRs!

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.

None yet

3 participants