Toastr sound

[csachdeva83]

• Issue: Add sound to notification #973
• Feature
• enableSound option in Individual Options
• Sound will be as per toastr type

[codecov]

Codecov Report

Base: 55.11% // Head: 55.49% // Increases project coverage by +0.37% 🎉

Coverage data is based on head (e5325d9) compared to base (efaf809).
Patch coverage: 50.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #978      +/-   ##
==========================================
+ Coverage   55.11%   55.49%   +0.37%     
==========================================
  Files          19       19              
  Lines         508      510       +2     
  Branches       88       89       +1     
==========================================
+ Hits          280      283       +3     
- Misses        216      217       +1     
+ Partials       12       10       -2     

Impacted Files	Coverage Δ
src/lib/toastr/toastr-config.ts	94.73% <ø> (ø)
src/app/home/home.component.ts	58.33% <50.00%> (-0.24%)	⬇️
src/lib/toastr/toastr.service.ts	51.04% <0.00%> (+2.08%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[csachdeva83]

@scttcper Please review this PR and list out any changes you desire.

[scttcper]

its interesting, but i don't think it belongs as part of the toastr

[csachdeva83]

its interesting, but i don't think it belongs as part of the toastr

Why so ?