adopted-ember-addons / adopted-ember-addons/ember-cli-flash

preventDuplicates w/ custom dedupe property

オープン
#337 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
TypeScript
スター
355
フォーク
118
PR マージ指標
30日以内にマージされた PR はありません

説明

I would like to be able to specify what key I want my flash messages to be de-duped on - sometimes I'll have flashes with the same `message`, but they'll have a different characteristic that means they should still be shown.

eg
```
{ type: 'warning', area: 'maths', message: 'please update your students' }
{ type: 'warning', area: 'reading', message: 'please update your students' }
{ type: 'warning', area: 'maths', message: 'please update your students' }
```

I think this could work well - providing a `dedupe` property
```
{ type: 'warning', area: 'maths', message: 'please update your students', dedupe: `${area}-${message}` }
{ type: 'warning', area: 'reading', message: 'please update your students', dedupe: `${area}-${message}` }
{ type: 'warning', area: 'maths', message: 'please update your students', dedupe: `${area}-${message}` } // should be de-duped
```

`dedupe` could default to `${message}`, Then this could be changed to `dedupe`
https://github.com/poteto/ember-cli-flash/blob/49aec796a5123481f9b546cd53409455f38c0c41/addon/flash/object.js#L12-L13

Would this be a welcome/useful addition? should I write a PR for it?

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。