fake_cloud_firestore icon indicating copy to clipboard operation
fake_cloud_firestore copied to clipboard

Published 20 hours ago • verified publisher icon atn832

mergeFields ignored in mock "set" method

Open EnduringBeta opened this issue 1 year ago • 2 comments

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.

EnduringBeta avatar May 15 '23 22:05 EnduringBeta

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!

atn832 avatar May 26 '23 06:05 atn832

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

EnduringBeta avatar May 26 '23 11:05 EnduringBeta