Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Export BrowserBugsnagStatic #1931

Closed
wants to merge 1 commit into from
Closed

Conversation

mdentremont
Copy link
Contributor

Goal

  • Allows the correct type to be imported in projects which use bugsnag

- Allows the correct type to be imported in projects which use bugsnag
@mdentremont
Copy link
Contributor Author

Our project had some code like this and we wanted to change trackInlineScript, but the "browser" exported bugsnag variable doesn't export its type:

    const bugsnagClientPromise = new Promise<BugsnagStatic>(
      (resolve) => {
        const bugsnag = require('@bugsnag/js');
        return resolve(bugsnag);
      },
    );

@johnkiely1
Copy link
Member

Hi @mdentremont, Thanks for the PR, we will look to review as soon as priorities allows.

@johnkiely1 johnkiely1 added the backlog We hope to fix this feature/bug in the future label Feb 27, 2023
@mdentremont
Copy link
Contributor Author

@johnkiely1 Any chance this'll get in soonish?

@johnkiely1
Copy link
Member

Hi @mdentremont, we don't have a definitive ETA as of yet but we are hoping to get to this soon.

@mdentremont
Copy link
Contributor Author

@johnkiely1 Any chance you'd like to sneak this in now?

@clr182
Copy link

clr182 commented Apr 5, 2024

Hi @mdentremont

Unfortunately we have no update as it stands. Once we have more information to share we will be sure to update this thread and inform you.

@djskinner
Copy link
Contributor

Superseded by #2112 (so we can run this through CI and get it merged)

@djskinner djskinner closed this Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog We hope to fix this feature/bug in the future
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants