Skip to content

Warn if childContextType is defined on SFC - #6933

Merged
jimfb merged 1 commit into
react:masterfrom
aweary:warn-child-context-sfc
Jun 8, 2016
Merged

Warn if childContextType is defined on SFC#6933
jimfb merged 1 commit into
react:masterfrom
aweary:warn-child-context-sfc

Conversation

@aweary

@aweary aweary commented Jun 1, 2016

Copy link
Copy Markdown
Contributor

Resolves #6926

Per the discussion there, this only warns for childContextTypes for now.

reactEurope-hackathon

@ghost

ghost commented Jun 1, 2016

Copy link
Copy Markdown

@aweary updated the pull request.

Add console.error message content check

Use appropriate name in warning/test
@aweary
aweary force-pushed the warn-child-context-sfc branch from 595ae2c to a303017 Compare June 1, 2016 10:12
@ghost

ghost commented Jun 1, 2016

Copy link
Copy Markdown

@aweary updated the pull request.

@jimfb

jimfb commented Jun 8, 2016

Copy link
Copy Markdown
Contributor

Everything is technically public API. cc @facebook/react-core

I like it, and Ben gave the conceptual signoff, so I'm shipping it. Also over a week old/reviewed/green.

@jimfb
jimfb merged commit c47830d into react:master Jun 8, 2016
@zpao zpao added this to the 15-next milestone Jun 8, 2016
zpao pushed a commit that referenced this pull request Jun 14, 2016
Add console.error message content check

Use appropriate name in warning/test
(cherry picked from commit c47830d)
@zpao zpao modified the milestones: 15-next, 15.2.0 Jun 14, 2016
aweary referenced this pull request Jul 5, 2016
(cherry picked from commit 74c29b3 and  bc1d59e)
gaearon added a commit that referenced this pull request Jul 5, 2016
zpao pushed a commit that referenced this pull request Jul 8, 2016
usmanajmal pushed a commit to usmanajmal/react that referenced this pull request Jul 11, 2016
mrizwanashiq pushed a commit to mrizwanashiq/react that referenced this pull request Jun 25, 2026
Add console.error message content check

Use appropriate name in warning/test
mrizwanashiq pushed a commit to mrizwanashiq/react that referenced this pull request Jun 25, 2026
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