Skip to content
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

feat: countDuplicates Show how many duplicates have stacked up so far #633

Merged
merged 22 commits into from
Mar 14, 2019

Conversation

scttcper
Copy link
Owner

see #530 for more details

@codecov

This comment has been minimized.

# Conflicts:
#	src/lib/toastr/toast-noanimation.component.ts
Repository owner deleted a comment from codecov bot Mar 13, 2019
@trevor-hackett
Copy link
Collaborator

Since this technically requires a title to be set, do we want to mention both in the example site and in the README that title is also required? Or do we somehow want to look into removing the title requirement?

@scttcper
Copy link
Owner Author

@yarrgh thats a good question. I think the way its implemented is ugly to the point where anyone using the feature would probably want to re-style and create their own toast. I think i'm fine putting that the toast puts it at the end of the title in the readme

@scttcper scttcper merged commit 6888c47 into master Mar 14, 2019
@scttcper scttcper deleted the pr/530 branch March 14, 2019 06:11
@scttcper
Copy link
Owner Author

🎉 This PR is included in version 10.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants