@arya-s is attempting to deploy a commit to the Sentry Team on Vercel.

A member of the Team first needs to authorize it.

Hey @arya-s - Thanks for the contribution! FYI, I'm going to wait until the code PRs are reviewed before taking a look at this.

Hey @arya-s - Thanks for the contribution! FYI, I'm going to wait until the code PRs are reviewed before taking a look at this.

Thanks!

Please let me know if there should be more documentation. As far as I've seen, onLoad also has no code samples so I orientated myself around that, but I'd be happy to add a snippet similar to the one I have in my sample project repo.

hey @arya-s, we've merged in the SDK and sentry changes. We can only merge in the docs once the SDK has been released, but I think adding a code snippet is a cool idea.

@shanamatthews, where do you think the best place to keep this content is?

The latest updates on your projects. Learn more about Vercel for Git ↗︎

hey @arya-s, we've merged in the SDK and sentry changes. We can only merge in the docs once the SDK has been released, but I think adding a code snippet is a cool idea.

@shanamatthews, where do you think the best place to keep this content is?

@AbhiPrasad @arya-s,

I think these docs are being moved and reworked in #8528, so for now, I think we can just put a code snippet on this page as an H3 subheading under "Customizing the Widget", as the last thing before "Feedback API"

Maybe something like "Using onLoad and onClose"?

What do you think?

I think these docs are being moved and reworked in #8528, so for now, I think we can just put a code snippet on this page as an H3 subheading under "Customizing the Widget", as the last thing before "Feedback API"

Maybe something like "Using onLoad and onClose"?

What do you think?

I had this in mind:

The optional callback handlers onLoad and onClose will be called when the widget opens or closes respectively. You can use these to run custom logic every time the widget is invoked and interacted with:

Sentry.showReportDialog({
  // ...
  onLoad() {
    // Log an event to amplitude when the report dialog opens
    amplitude.logEvent('report_dialog_seen');
  },
  onClose() {
    // Refresh the page after the user closes the report dialog
    location.reload();
  }
});

But it's going to take me a bit to write snippets for all the supported languages/frameworks, some of which I don't have too much experience in and don't have them running locally to test the code... 😅

Thoughts?

I had this in mind:

The optional callback handlers onLoad and onClose will be called when the widget opens or closes respectively. You can use these to run custom logic every time the widget is invoked and interacted with:

Sentry.showReportDialog({
  // ...
  onLoad() {
    // Log an event to amplitude when the report dialog opens
    amplitude.logEvent('report_dialog_seen');
  },
  onClose() {
    // Refresh the page after the user closes the report dialog
    location.reload();
  }
});

But it's going to take me a bit to write snippets for all the supported languages/frameworks, some of which I don't have too much experience in and don't have them running locally to test the code... 😅

Thoughts?

The snippet looks good to me! @AbhiPrasad - thoughts?

I would say just create code snippets for the language(s) you're comfortable with. We can wrap this section in a <PlatformSection supported={["<languages>"]}> and recruit SDK engineers to create other samples for their respective languages once this is merged.

The snippet looks good to me!

Looks good to me too, let's make that change!

@shanamatthews @AbhiPrasad
I added snippets for js and python.

Regarding the exact SDK version I wasn't sure. I'm assuming the onClose feature will go out with v7.82.0 (next minor) but if not, please change or let me know and I can change it.