Skip to content

let the user define a global dismissAfter prop (dismiss is now handled by the component, not the action anymore)#33

Open
Grsmto wants to merge 7 commits intoindexiatech:masterfrom
Grsmto:set_global_dissmiss_time
Open

let the user define a global dismissAfter prop (dismiss is now handled by the component, not the action anymore)#33
Grsmto wants to merge 7 commits intoindexiatech:masterfrom
Grsmto:set_global_dissmiss_time

Conversation

@Grsmto
Copy link
Copy Markdown

@Grsmto Grsmto commented Jul 20, 2016

Hey,
It doesn't seems like I can re-open the previous PR #8.
I updated the PR with the latest version of Master.
You can test in the demo (just change the dismiss time to any value).

Thanks!

@Grsmto Grsmto changed the title Set global dissmiss time let the user define a global dismissAfter prop (dismiss is now handled by the component, not the action anymore) Jul 20, 2016
@kylecesmat kylecesmat closed this Sep 11, 2016
@Grsmto
Copy link
Copy Markdown
Author

Grsmto commented Sep 11, 2016

:/

@kylecesmat
Copy link
Copy Markdown

kylecesmat commented Sep 11, 2016

@Grsmto sorry - We can reopen this PR. I assumed it was inactive, my apologies. If you'd like to refresh your branch we can actually merge this.

@kylecesmat kylecesmat reopened this Sep 11, 2016
@Grsmto
Copy link
Copy Markdown
Author

Grsmto commented Sep 11, 2016

Ok no worries! Just tell me if you're still interested by the feature and I can update again :p As you want!

@kylecesmat
Copy link
Copy Markdown

@Grsmto I think it's a great feature! This repository has just been dormant so hard to regain context of what is going on. My apologies again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants