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.

Sign up for GitHub

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

Jump to bottom

[RN] Consolidate eventTypes registry with view configs #12556

Merged
merged 1 commit into from
Apr 10, 2018
Merged

[RN] Consolidate eventTypes registry with view configs #12556

merged 1 commit into from
Apr 10, 2018

Commits on Apr 10, 2018

  1. Consolidate eventTypes registry with view configs 

    We already have one stateful module that contains all the view config.
We might as well store the event types there too. That way the shared
state is compartmentalized (and I can move it out in a follow up PR).

The view config registry also already has an appropriate place to call
processEventTypes so now we no longer have to do it in RN.

Will follow up with a PR to RN to remove that call.
    @sebmarkbage
    sebmarkbage committed Apr 10, 2018
    Configuration menu
    Copy the full SHA
    ce7bbd6 View commit details
    Browse the repository at this point in the history