Skip to content

[modularization] Move FilterView to a new module#2896

Merged
rye merged 6 commits into
masterfrom
reorg-take6-part5.13
Aug 27, 2018
Merged

[modularization] Move FilterView to a new module#2896
rye merged 6 commits into
masterfrom
reorg-take6-part5.13

Conversation

@hawkrives

@hawkrives hawkrives commented Aug 26, 2018

Copy link
Copy Markdown
Member

Moves the "filter" components into a module, @frogpond/filter.

Part of the great #1537 redo.

Closes #2895. (because it includes it)

@hawkrives hawkrives self-assigned this Aug 26, 2018
@hawkrives hawkrives requested a review from a team August 26, 2018 21:43
rye
rye previously approved these changes Aug 26, 2018
drewvolz
drewvolz previously approved these changes Aug 26, 2018
@rye rye dismissed stale reviews from drewvolz and themself via e1e28fe August 26, 2018 22:42
@rye rye force-pushed the reorg-take6-part5.13 branch from c4bb9ce to e1e28fe Compare August 26, 2018 22:42
@rye

rye commented Aug 26, 2018

Copy link
Copy Markdown
Member

This PR could use a sanity check post-rebase.

@hawkrives

Copy link
Copy Markdown
Member Author

So if this doesn't include the commits from #2893, how can https://github.com/StoDevX/AAO-React-Native/pull/2896/files#diff-40d4f57836a110fe35c2488ee1c1a70bR17 work?

@hawkrives

Copy link
Copy Markdown
Member Author

Needs #2893

@rye

rye commented Aug 26, 2018

Copy link
Copy Markdown
Member

Also needed #2894 and #2895. Flow is happy locally, so this should pass.

@rye rye force-pushed the reorg-take6-part5.13 branch from 9a1aa63 to f647a07 Compare August 27, 2018 00:43

@rye rye left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Atomic PR! Yay!

@rye rye merged commit d2d19ab into master Aug 27, 2018
@rye rye deleted the reorg-take6-part5.13 branch August 27, 2018 01:51
@rye rye added this to the v2.6 milestone Sep 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants