-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
Updating the documentation. #329
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
Comments
@okaris I think the S3Adapter was merged a bit prematurely as there is currently no test coverage.I wouldn't feel comfortable using it in production. Push support is still open: #311. I think if you're looking to run this in any kind of customer facing environment at the moment you need to be prepared to get your hands dirty. But, I agree there should be some running road map and versioning planning. |
+1 |
We also haven't published a new version on NPM containing a lot of these changes. We'll make sure to better document these things before pushing version 2.0.8 |
I believe the documentation in the ReadMe file needs to be updated as new features are added and the existing are changed. I see a lot of people, including myself, wondering how to use the new Push feature added, or how to use S3 as the default file storage. I feel bad not being able to contribute but just giving opinion from the outside. Also is there a way to know who is working on what and what else needs working on, so maybe I can find something to help with. Please do keep up the good work!
The text was updated successfully, but these errors were encountered: