Use platform specific imports for Rollbar.js - Githubissues

rollbar / rollbar-react-native

Crash and error reporting for React Native using Rollbar

https://docs.rollbar.com/docs/react-native

62 stars 37 forks source link

Use platform specific imports for Rollbar.js #172

Closed waltjones closed 1 year ago

waltjones commented 1 year ago

Description of the change

Updates import of Rollbar.js to be platform specific.
Updates Typescript type definitions to match the new Client interface
The file containing the main Client class is renamed from Rollbar.js to Client.js, to avoid confusion with the Rollbar.js package.

Note: The file rename causes an awkward PR diff for ./Client.js and ./Rollbar.js. The commit diff is much easier to read -> 7f0ce5ab6306d81c6a9471a4d03ad2cc029d438c

Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

Fixes

Code review

[x] This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[x] Issue from task tracker has a link to this pull request
[x] Changes have been reviewed by at least one other engineer