Angularfire: Allow to perform set with additional parameters (e.g. { merge: true })

[iclems]

https://github.com/angular/angularfire2/blob/21133852c2cfcb2fc1402e63e8cddc0cff70d15c/src/firestore/document/document.ts#L52

[davideast]

@iclems Want to send in the PR? :)

[iclems]

Sure! I always start by opening tickets to remember it myself ;-)

[codediodeio]

On a related note, it looks like these methods require a plain object.

Is this expected or should the methods look like this?

return this.ref.update({ ...data });

[bergergit]

Any updates on this?
Being able to use { merge: true } is crucial

[jtomek]

Indeed this is a crucial feature.

var cityRef = db.collection('cities').doc('BJ');

var setWithMerge = cityRef.set({
    capital: true
}, { merge: true });

https://cloud.google.com/firestore/docs/manage-data/add-data

EDIT: It is possible to use the firebase ref endpoint:

this.afs.collection(`users/${variable}/collection`).doc("yourid").ref
 .set({
  capital: true
 }, { merge: true });

[hiepxanh]

hello, is that problem resolved and we just only need update @jtomek solution to docs?

[codediodeio]

This issue has already been fixed with #1332. No need to use the workaround above. You can do it like this.

cityRef.set({ capital: 'sactown' }, { merge: true });

[hiepxanh]

Yahoo! one more problem resolved. Can we close that issue @iclems 😄

[jamesdaniels]

Fair. There's a work around with the new API now. Closing due to it being outdated.