-
Notifications
You must be signed in to change notification settings - Fork 0
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
TypeScript reformat #9
Comments
Will do, thank you! |
And you must create this React 18 friendly, this React relased 2 days ago but on refactor it's required :) If u looking for contributor i can help you, but i typing in functions react, IMO it's better way :) I have a idea for 1 next feature, deleting items not from selected but from e.p. Data Base this is must have, in my current project i want component like this, but with editting, adding and deleting My idea is add a button on list item like trash this button will deleting item hard, this will be a next callback. |
We can refactor whole code, and add new features :) |
2 more suggestions, on Example code, don't give console.log and demo didn't work |
yes functional component sounds great too. We have to reformat the whole code. Let's define what changes we want, then we can choose the feature we want to work on. |
yup, it's been crashing on codesandbox ever since I released v 2.1.0 Regarding console.log, I added it so that user knows what's happening with the lists. |
I start to defining features here: https://github.com/PatrykKuniczak/Notification_Manager/issues/9 |
Yeah, but user can do it if want, i don't see a lot of console logs in docs overall |
Hi
Your package looks good, but i want to use it in TypeScript project, can u create a update, which allows to use it with TS.
A lot of people coding in TS, because it's very useful :)
The text was updated successfully, but these errors were encountered: