Skip to content
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

Date now converts to overridden type #34

Closed
wants to merge 3 commits into from
Closed

Date now converts to overridden type #34

wants to merge 3 commits into from

Conversation

jon49
Copy link
Contributor

@jon49 jon49 commented Aug 13, 2015

Also, fixed spelling of overriden -> overridden

An alternative to this would be just to bypass $ref whenever there is an overridden type.

It still throws the error. It happens at:
https://github.com/lbovet/typson/blob/master/lib/typson.js#L86

Not sure what solution should be used to silence the error. Could check if the value in semanticDiagnostics is in the array overriddenTypes. But I would need to create a new file that both files can call. Of course, we could just let the error pass through, just annoying though when a person is using error checking.

fixes #16

jon49 and others added 3 commits August 13, 2015 12:36
Also, fixed spelling of overriden -> overridden

An alternative to this would be just to bypass `$ref` whenever there is an overridden type.

It still throws the error. It happens at:
https://github.com/lbovet/typson/blob/master/lib/typson.js#L86

Not sure what solution should be used to silence the error. Could check if the value in `semanticDiagnostics` is in the array `overriddenTypes`. But I would need to create a new file that both files can call. Of course, we could just let the error pass through, just annoying though when a person is using error checking.

fixes #16
Buffer is sometimes used for mongoose schema. It is nice to be able to have multiple types on this one.
@jon49 jon49 closed this by deleting the head repository Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Could not find symbol Date
2 participants