Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

fromJson JSDoc #3070

Closed
pschulten opened this issue Jun 26, 2013 · 1 comment
Closed

fromJson JSDoc #3070

pschulten opened this issue Jun 26, 2013 · 1 comment
Assignees
@pkozlowski-opensource

Comments

@pschulten
Copy link

The JSDoc from the fromJson function states that it:
* @returns {Object|Array|Date|string|number} Deserialized thingy.
but it does not return Dates
@pkozlowski-opensource
Copy link
Member

@pschulten Indeed. Would you like to send us a pull request with the documentation update?

@pkozlowski-opensource pkozlowski-opensource self-assigned this Feb 23, 2014
linclark added a commit to linclark/angular.js that referenced this issue Feb 27, 2014
@linclark
docs(core): update fromJson return values
67e4881 
Dates are not returned, so remove Date from list of return values.

Closes angular#3070
@linclark linclark mentioned this issue Feb 27, 2014
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@pkozlowski-opensource pkozlowski-opensource

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs(core): update fromJson return values linclark/angular.js
2 participants
@pkozlowski-opensource @pschulten