Centralize `SentryInAppLogic` in a singleton

Open armcknight opened this issue 3 years ago • 2 comments

We create 6 separate instances but they all seem to use the same list of includes/excludes from SentryOptions, and then those are passed around in various ways like this. This can be refactored in a separate PR.

Originally posted by @armcknight in https://github.com/getsentry/sentry-cocoa/pull/2724#discussion_r1127256094

Mar 08 '23 04:03 armcknight

Great catch; I also came across this recently, but I didn't create an issue. Thanks 🙏

Mar 14 '23 10:03 philipphofmann

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

Apr 05 '23 00:04 github-actions[bot]