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

docs(filter.js): documents second parameter to .register #12378

Closed
wants to merge 1 commit into from

Conversation

matthewjh
Copy link
Contributor

Is there any reason the second parameter of $FilterProvider#register wasn't documented? If not, here.

@lgalfaso
Copy link
Contributor

I do not expect a lot of people to actually use this API, but module.filter, anyhow, thanks for the PR.

@matthewjh
Copy link
Contributor Author

Agreed Lucus. I used it for the first time with typescript and the definition was off (it spoke nothing of a second parameter), presumably because of this doc.

On Sun, Jul 19, 2015 at 5:52 AM -0700, "Lucas Mirelmann" [email protected] wrote:
I do not expect a lot of people to actually use this API, but module.filter, anyhow, thanks for the PR.


Reply to this email directly or view it on GitHub:
#12378 (comment)

netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
ggershoni pushed a commit to ggershoni/angular.js that referenced this pull request Sep 29, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants