-
Notifications
You must be signed in to change notification settings - Fork 51
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
setup raven.js #22
Comments
Second option is basically working already.. ember-cli-sentry and ember-cli-deploy-sentry work along just fine together, it's just that ember-cli-sentry is not a required depdendency |
Yes, I understand, yet my point was to highlight that some/all of the apps that use |
The two projects work side-by-side, but do not work together without further configuration. This project sets the application revision in a meta tag, but ember-cli-sentry does not use the meta tag, and sends reports with a different revision. I think this worked in the past, but it does not work today. |
@RobinDaugherty check out #39 as a proposal to get rid of the meta tag generally. |
From the README:
I think this would be a great addition to the addon as it would allow a simpler setup and overall a better experience of Sentry+Ember.
If it appears relevant to the community, I can guess two options:
ember-cli-deploy-sentry
andember-cli-sentry
ember-cli-sentry
as a dependency forember-cli-deploy-sentry
I'd be more than happy to discuss the options.
The text was updated successfully, but these errors were encountered: