PostHog / posthog-js-lite

Reimplementation of posthog-js to be as light and modular as possible.
https://posthog.com/docs/libraries
MIT License
69 stars 36 forks source link

fix: only logging if debug enabled #284

Closed marandaneto closed 1 month ago

marandaneto commented 1 month ago

Problem

Closes https://github.com/PostHog/posthog-react-native-session-replay/issues/7

Changes

Only logs message if debug is enabled

Release info Sub-libraries affected

Bump level

Libraries affected

Changelog notes

github-actions[bot] commented 1 month ago

Size Change: +191 B (+0.18%)

Total Size: 105 kB

Filename Size Change
posthog-node/lib/index.cjs.js 20 kB +29 B (+0.15%)
posthog-node/lib/index.esm.js 20 kB +29 B (+0.15%)
posthog-react-native/lib/posthog-core/src/index.js 9.72 kB +32 B (+0.33%)
posthog-react-native/lib/posthog-react-native/src/posthog-rn.js 3.8 kB +36 B (+0.96%)
posthog-react-native/lib/posthog-react-native/src/version.js 123 B +1 B (+0.82%)
posthog-web/lib/index.cjs.js 17.1 kB +32 B (+0.19%)
posthog-web/lib/index.esm.js 17.1 kB +32 B (+0.19%)
ℹ️ View Unchanged | Filename | Size | | :--- | :---: | | `posthog-react-native/lib/posthog-core/src/eventemitter.js` | 1.08 kB | | `posthog-react-native/lib/posthog-core/src/lz-string.js` | 1.42 kB | | `posthog-react-native/lib/posthog-core/src/types.js` | 365 B | | `posthog-react-native/lib/posthog-core/src/utils.js` | 822 B | | `posthog-react-native/lib/posthog-core/src/vendor/uuidv7.js` | 2.04 kB | | `posthog-react-native/lib/posthog-react-native/index.js` | 485 B | | `posthog-react-native/lib/posthog-react-native/src/autocapture.js` | 1.78 kB | | `posthog-react-native/lib/posthog-react-native/src/frameworks/wix-navigation.js` | 651 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useFeatureFlag.js` | 437 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useFeatureFlags.js` | 362 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useLifecycleTracker.js` | 416 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useNavigationTracker.js` | 628 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/usePostHog.js` | 249 B | | `posthog-react-native/lib/posthog-react-native/src/legacy.js` | 810 B | | `posthog-react-native/lib/posthog-react-native/src/native-deps.js` | 1.17 kB | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalAsyncStorage.js` | 183 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoApplication.js` | 215 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoDevice.js` | 211 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoFileSystem.js` | 224 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoLocalization.js` | 216 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalReactNativeDeviceInfo.js` | 220 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalReactNativeNavigation.js` | 218 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalReactNativeNavigationWix.js` | 222 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalSessionReplay.js` | 231 B | | `posthog-react-native/lib/posthog-react-native/src/PostHogContext.js` | 210 B | | `posthog-react-native/lib/posthog-react-native/src/PostHogProvider.js` | 1.49 kB | | `posthog-react-native/lib/posthog-react-native/src/storage.js` | 1.09 kB | | `posthog-react-native/lib/posthog-react-native/src/types.js` | 90 B |

compressed-size-action

marandaneto commented 1 month ago

PostHog/posthog-js@main/src/utils/logger.ts

could do something like that but not blocking :)

yep and it'd help with https://github.com/PostHog/posthog-js-lite/issues/113 but focusing on a quick fix for now