Denis Andrašec
Denis Andrašec
> can those commands be run in production? Testes in debug in am Windows VM and tests pass so far.
@vaind Hey! Just started this and i'm seeing that `example_web_compile_test.dart` fails, as a file in the package uses dart:io https://github.com/dart-lang/core/blob/main/pkgs/platform/lib/src/interface/local_platform.dart Not sure how to circumvent this. Any suggestions?
@vaind Pushed the code i had locally. Pretty much replaced our `Platform` definition in favour of `package:platform/platform.dart`, but here we have the issue that `dart:io` is used.
@vaind Thank you for your insights and the PR, i'll take a look at it. 🙇
I'm not sure regarding naming of the "old" `SentryLog` classes. Another possibility would be `SentryLogging`. Would not go with `SentryLogger`, as the new Logs API already has `Sentry.logger.warn(...)` as naming,...
@philipphofmann Thank you, no worries. I'll split them up.
@philipphofmann Implemented handling of the case where envelope to data fails. Waiting for CI so we can do a final review/feedback round.
@fryette Hey! We recently released [2.4.1](https://github.com/getsentry/sentry-dart-plugin/releases/tag/2.4.1), which should address issues with source context.
@otopba Could you check if everything works as expected on sentry.io? So is source context present in your issues where enabled with 2.4.1? I got the same warnings when running...
Ok, then we need to figure out why we report this warning when everything is on order. Thx for checking.