adopted-ember-addons / adopted-ember-addons/validated-changeset
Snapshot changes is wrong when object has proxy attributes
- 主要言語
- TypeScript
- スター
- 38
- フォーク
- 27
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
Hi, changeset of ED Models with async belongsTo relationships show incorrect changes on snapshot:

`change` and `changes` are correct, but `snapshot().changes` shows all my async belongsTo as changed attributes incorrectly.
I think this is because of this unrwrap https://github.com/validated-changeset/validated-changeset/blob/da68651a41028ab827872cf10960920779c2a09f/src/index.ts#L975
Which adds all the unwrapped relationships to `this[CHANGES]`. When i get `change` and `changes` these incorrect changes are filtered out, but not on snapshot.
Does my explanation make sense? Thanks in advance!
EDIT:
If anyone else comes here looking for a solution I'm currently creating snapshots like this until it is fixed
```js
function createSnapshot(cs) {
const changes = {};
cs.changes.forEach(({ key, value }) => { changes[key] = value; });
return {
changes,
errors: { ...cs.error },
};
}
```
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
評価
この issue はまだ評価されていません。