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

feat(common): introduce KeyValuePipe #24319

Closed
wants to merge 6 commits into from

fixup! feat(common): introduce KeyValuePipe

  • Loading branch information...
Toxicable committed Jun 13, 2018
commit b99eba1f27c3241800a0ac23c5d4d34f55f5ca18
@@ -36,7 +36,7 @@ export interface KeyValue<K, V> {
* This examples show how an Object or a Map and be iterated by ngFor with the use of this keyvalue
* pipe.
*
* {@example common/pipes/ts/keyvalue.ts region='KeyValuePipe'}
* {@example common/pipes/ts/keyvalue_pipe.ts region='KeyValuePipe'}
*/
@Pipe({name: 'keyvalue', pure: false})
export class KeyValuePipe implements PipeTransform {

This comment has been minimized.

Copy link
@alfaproject

alfaproject Jun 13, 2018

Contributor

Any reason not to do KeyValuePipe<K, V>? It down-levels to the exact same code and would avoid having to use any, I think.

ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.