Skip to content
This repository has been archived by the owner on Oct 15, 2019. It is now read-only.

add plugin support to sentry #1

Open
ndhoule opened this issue Jun 11, 2015 · 8 comments
Open

add plugin support to sentry #1

ndhoule opened this issue Jun 11, 2015 · 8 comments
Labels
migrated The issue has been migrated

Comments

@ndhoule
Copy link
Contributor

ndhoule commented Jun 11, 2015

From @ianstormtaylor on June 7, 2014 1:21

Sentry lets you add plugins when loading their library, would be good to let the user pass in which plugins they'd like to load.

Copied from original issue: segmentio/analytics.js-integrations#197

@ndhoule
Copy link
Contributor Author

ndhoule commented Jun 11, 2015

From @jivinivan on December 1, 2014 19:16

👍

@hankim813
Copy link
Contributor

looked up sentry docs for how to do this but can't find any examples of this. closing due to inactivity + lack of context here.

@QuentinFchx
Copy link

QuentinFchx commented Apr 25, 2016

https://docs.getsentry.com/hosted/clients/javascript/integrations/
Example for angular
I think it should be something along:

<% plugins = ['angular', 'console'] %>
<script src="https://cdn.ravenjs.com/2.3.0/<%= plugins.join(',') %>/raven.min.js"></script>
<!-- instead of -->
<script src="https://cdn.ravenjs.com/2.3.0/raven.min.js"></script>

For Angular, the main issue is that users have to make sure that raven+angular is loaded before bootstraping their app.

@hankim813
Copy link
Contributor

Thanks! I'll reopen it and triage this issue internally -- happy to accept PRs in the meantime though! :)

@hankim813 hankim813 reopened this Apr 25, 2016
@xavhan
Copy link

xavhan commented Aug 3, 2017

up again,
Seems that Raven CDN exposes their plugins as well at http://ravenjs.com/
hope this could help.

@xavhan
Copy link

xavhan commented Aug 3, 2017

If instead of loading https://cdn.ravenjs.com/3.17.0/raven.min.js you provide the possibility to load
https://cdn.ravenjs.com/3.17.0/angular/raven.min.js it would make the use of sentry and segment within an Angular app seemless ;)

@hankim813
Copy link
Contributor

cc @anoonan

@SegmentDestinationsBot
Copy link
Contributor

Hi @ndhoule, as part of the monorepo migration, this issue has been moved to new issue. Our engineers have been notified and will prioritize and work on it ASAP. Thank you!

For more information, see README.md.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
migrated The issue has been migrated
Projects
None yet
Development

No branches or pull requests

5 participants