• Changed Files
  • packages/core/src/operators/dedupe.ts

[changed] dedupe to multicast rather than remember
ClosedPublic

Authored by appsforartists on Jul 24 2017, 8:42 PM.

Details

Summary

dedupe needs to be multicasted to work correctly. This used to be only achievable with _remember(), but we have proper _multicast() now, so let's use it.

Diff Detail

Repository
R13 material-motion/material-motion-js
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
appsforartists created this revision.Jul 24 2017, 8:42 PM
Restricted Application added a reviewer: O2: Material Motion. · View Herald TranscriptJul 24 2017, 8:42 PM
Restricted Application added a project: Material Motion. · View Herald Transcript
Restricted Application added a reviewer: O3: Material JavaScript platform reviewers. · View Herald Transcript
Restricted Application added a reviewer: Material Motion. · View Herald Transcript
Restricted Application failed to build Restricted Buildable!Jul 24 2017, 9:13 PM

Rebased on fixed CI config

Restricted Application failed to build Restricted Buildable!Jul 24 2017, 9:22 PM

Rebased on fixed draggable enabled

Restricted Application completed building Restricted Buildable.Jul 24 2017, 9:36 PM
featherless accepted this revision.Jul 27 2017, 4:48 PM
This revision is now accepted and ready to land.Jul 27 2017, 4:48 PM
This revision was automatically updated to reflect the committed changes.