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

Updating to React v15.6.1. #4057

Merged
merged 1 commit into from Aug 4, 2017

Conversation

Projects
None yet
2 participants
@dennisoelkers
Member

dennisoelkers commented Aug 3, 2017

This change is bumping React to the most versions. General functionality was tested, there should be no semantical changes. We have new warnings in the console (mostly about https://fb.me/react-warning-dont-call-proptypes) which will be remediated in later pull requests.

@edmundoa

This comment has been minimized.

Member

edmundoa commented Aug 4, 2017

I also see lots of deprecations being logged for isMounted. After #4059, I think that should be the next thing to tackle.

@edmundoa

LGTM 👍

@edmundoa edmundoa merged commit 16b3659 into master Aug 4, 2017

4 checks passed

ci-web-linter Jenkins build graylog-pr-linter-check 1843 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
graylog-project/pr Jenkins build graylog-project-pr-snapshot 360 has succeeded
Details

@edmundoa edmundoa deleted the react-upgrade-15.6.1 branch Aug 4, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment