-
Notifications
You must be signed in to change notification settings - Fork 27.4k
docs(CHANGELOG): add info about $parse breaking change in 1.3.3 #10131
docs(CHANGELOG): add info about $parse breaking change in 1.3.3 #10131
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/. If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits. Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name. |
Oops, we did it again - breaking change. ;) I think we should add the info about naming filters to https://docs.angularjs.org/guide/filter and https://docs.angularjs.org/api/ng/provider/$filterProvider too |
It would be nice to throw when trying to register a filter that is not an |
CLA signature verified! Thank you! Someone from the team will now triage your PR and it will be processed based on the determined priority (doc updates and fixes with tests are prioritized over other changes). |
@Narretz I would be too keen on documenting it everywhere - the fact that it was supported was really an accident, IMO |
CLAs look good, thanks! |
@mary-poppins this is awkward... we actually don't need you anymore |
Hey, don't deprecate @mary-poppins for some ...bot !!! |
@pkozlowski-opensource - this looks good but I agree that it should documented - but in a positive sense. Something along the lines of:
|
@petebacondarwin are you referring to some additional documentation on a page dedicated to filters? Or rewording the breaking change section text? |
Extra docs on filter api page
|
1b919af
to
d604d94
Compare
OK, added a note to the filters guide as well. |
Closes #10054