TryCatchLearn / ReventsAlpha

Update to the content that uses React-redux 7.1.0-alpha.4, 3.0.0-alpha.12 and an excessive use of react hooks...
17 stars 10 forks source link

Remove react-redux-firebase dependancy from the project #1

Open TryCatchLearn opened 5 years ago

TryCatchLearn commented 5 years ago

I think this course will be better if we used our own reducers rather than relying on a 3rd party for this. This will make us immune from breaking changes in 3rd party dependancies such as this and future proof what we are doing here.

jphyqr commented 5 years ago

Awesome update. I used the Revents project as a base for a web app I have worked on for the last few months. I have come back to this course many times when I needed help. After seeing the Hooks API, I wanted to start a new project that used hooks. I came back here just to watch how you initialize a project, and what do you know, you have shown how to use hooks! I cannot explain how awesome this is for the student. It is one thing to learn something new like hooks, and another thing to learn it within the framework of a project we already are comfortable with. Understanding the Events detailed page, mapStateToProps, useFirestore, isEmpty, etc, makes learning the useSelector hook NOTHING. Since your course seems to resonate with my learning path so well, I have to make a suggestion: What about making a lightweight React Native version of revents? If I was to learn how to make a native app, I would surely want to learn it at this pace! There seems to be some pretty cool Google Cloud features that are more for onboarding to mobile (deep linking, google cloud messaging, etc) that would probably work well with this project in a react-native environment. Or-- something that uses the Machine Learning kit? If you made any spawn projects off of revents I would be the first to sign up!

Gagan-97 commented 5 years ago

What was the outcome of your analysis? Is the boilerplate code too much or it can be adjusted to futureproof the app?

panoscool commented 4 years ago

I would suggest on this issue to remove redux-firestore as well for the same reasons as the above mentioned.