search

atn832 / fake_cloud_firestore

BSD 2-Clause "Simplified" License
118 stars 88 forks source link

mergeFields ignored in mock "set" method #269

Open EnduringBeta opened 1 year ago

EnduringBeta commented 1 year ago

Please let me know if I'm mistaken here, but after looking at the source code, I felt confident enough to report.

In the SetOptions docs, mergeFields is a valid argument, but in the equivalent mocked code, it's not used.

If I'm not misunderstanding, then this may just be a feature request.

atn832 commented 1 year ago

Thanks for reporting this issue @EnduringBeta. You're right, this feature is not implemented so far. If you are willing to propose a PR, I'll be happy to review it!

EnduringBeta commented 1 year ago

I don't have the bandwidth to currently do it myself, but I do appreciate the confirmation about the status. Thanks.