Add setRecord method to LogEntryEventBuilder for handling Set of Ids and List of Ids

jongpie / NebulaLogger

The most robust observability solution for Salesforce experts. Built 100% natively on the platform, and designed to work seamlessly with Apex, Lightning Components, Flow, Process Builder & integrations.

https://nebulalogger.com

MIT License

708 stars 164 forks source link

Add setRecord method to LogEntryEventBuilder for handling Set of Ids and List of Ids #792

Closed alanjaouen closed 2 weeks ago

alanjaouen commented 3 weeks ago

As LogEntryEventBuilder provide methods setRecord(SObject record) and setRecord(List<SObject> records), I sugest to add setRecord(Set<Id> recordsIds) to complete setRecord(Id recordId)

I reused setRecord(List<SObject> records) to avoid duplicating code, but feel free to give me your though on this

alanjaouen commented 2 weeks ago

thanks for your feedback

please rebase before merging to squash commits together

jamessimone commented 2 weeks ago

Nice, thanks!

jongpie commented 2 weeks ago

Alright, this is now available in release v4.14.18! Thanks again @alanjaouen for this idea/PR, I think this is a really great enhancement 🎉 🥳