-
-
Notifications
You must be signed in to change notification settings - Fork 934
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
docs: updating README to the new flame bloc version #1737
Conversation
cc @felangel |
@@ -1,6 +1,6 @@ | |||
MIT License | |||
|
|||
Copyright (c) 2021 Blue Fire |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -0,0 +1,55 @@ | |||
# Flame Bloc 🔥🧱 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps add a comment here about when this deprecated readme can be removed. This way we make sure the file won't linger forever.
[this](./DEPRECATED_README.md). | ||
|
||
For a migration guide from the previous API to the current one, | ||
[check this article](https://verygood.ventures/blog/flame-bloc-new-api). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
updateGear(state); | ||
} | ||
} | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might want to call out one limitation of the mixin is that a component cannot listen to multiple blocs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, once the requested changes from Felix are applied!
And I don't think that we should keep the deprecated readme file, potentially link to the old tagged version instead if you want the user to find it (but I don't think that is necessary either).
packages/flame_bloc/README.md
Outdated
|
||
## ⚠️ IMPORTANT ⚠️ ## | ||
This README only covers the new (>= 1.4.0) API of flame bloc, for the previous deprecated API check | ||
[this](./DEPRECATED_README.md). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we need this, if they want to read an old version of the readme they can look at the tag for an old version.
Co-authored-by: Felix Angelov <[email protected]>
Description
Updates the README to match the new API.
Checklist
fix:
,feat:
,docs:
etc).docs
and added dartdoc comments with///
.examples
.Breaking Change
Related Issues
Fixes #1695