Don't set lastEventId for transactions

[romtsn]

lastEventId could be used for reporting a userfeedback, so we cant assign the lastEventId for transactions

Spec: https://develop.sentry.dev/sdk/performance/#hub-changes

See getsentry/develop#431 for more details

• Java
• Cocoa
• Dart
• .NET
• React Native? @marandaneto
• Unity?

[brustolin]

Cocoa don't have lastEventId.
Is this something we should have?

[romtsn]

we set it for captureEvent captureException captureMessage, I guess thats where we could add it, if not yet

[krystofwoldrich]

RN was missing the function. It was accessible only indirectly thru the hub. I'm waiting for a new JS release to test the feature. But we don't change the JS logic and that one is correct.

[kahest]

we set it for captureEvent captureException captureMessage, I guess thats where we could add it, if not yet

@philipphofmann @brustolin Do we need this?

[kahest]

RN was missing the function. It was accessible only indirectly thru the hub. I'm waiting for a new JS release to test the feature. But we don't change the JS logic and that one is correct.

@krystofwoldrich Have you been able to test this?

[brustolin]

@philipphofmann @brustolin Do we need this?

We probably do.

[krystofwoldrich]

@kahest Yes, it works correctly, ot doesn't set the id for transactions.

[kahest]

@philipphofmann @brustolin Do we need this?

We probably do.

Created getsentry/sentry-cocoa#2705 for discussion.

Closing this issue since everything is done. 👍