callstack / react-native-material-palette

Bringing Material Palette API to React Native
MIT License
104 stars 11 forks source link

Minor refactor #35

Closed satya164 closed 7 years ago

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling e305df53811dba048467c53d0a724495a8ce78d0 on @satya164/tweaks into 102e55bd28905f98e82012294229bbb9036d9912 on master.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 081f1c896b2eb3e343888701708b1fb3974cf690 on @satya164/tweaks into 102e55bd28905f98e82012294229bbb9036d9912 on master.

rgommezz commented 7 years ago

Also, we need to update the API docs to remove onInit from there

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 589b7ae6f9dd6b3e4f50b282c30c3b7280018384 on @satya164/tweaks into 102e55bd28905f98e82012294229bbb9036d9912 on master.

satya164 commented 7 years ago

@zamotany it's basically the same as when the provider mounts, so seems unnecessary