This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Unify proptypes in util/proptypes.js #3728

Merged

jacogr merged 2 commits into master from jg-proptypes

Dec 7, 2016

Contributor

jacogr commented Dec 6, 2016 •

edited

Loading

combine all defined proptypes in util/proptypes.js
add new PropTypes as cherry-picked from WIP branches
change all existing references
make references consistent between nullable & nullableProptype


          Unify proptypes in util/proptypes.js

aae1994

jacogr added A0-pleasereview 🤓 M7-ui labels

Contributor

derhuerst commented Dec 6, 2016

There are a few missing:

ui/ConfirmDialog/confirmDialog.js
ui/Container/container.js
ui/Container/Title/title.js
ui/Form/InputInline/inputInline.js
ui/Modal/modal.js
ui/Modal/Title/title.js


          Add missing use of nodeOrStringProptype

0a4fd46

derhuerst added A8-looksgood 🦄 and removed A0-pleasereview 🤓 labels

jacogr mentioned this pull request

Add functionalities to multi-sig wallet #3729

Merged

jacogr merged commit e2bb8ef into master

jacogr deleted the jg-proptypes branch

December 7, 2016 12:50

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