-
Notifications
You must be signed in to change notification settings - Fork 20
Update Android to use Sentry org auth token #651
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
Update Android to use Sentry org auth token #651
Conversation
🛰️ Build DistributionBuild available for installation
🛸 Powered by Emerge Tools |
android/app/build.gradle.kts
Outdated
|
|
||
| sentry { | ||
| org.set("emerge-tools") | ||
| authToken.set(System.getenv("SENTRY_SENTRY_AUTH_TOKEN")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not all of the workflows set SENTRY_SENTRY_AUTH_TOKEN
It looks like some do this:
SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
and some do
SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_SENTRY_AUTH_TOKEN }}
We should consolidate them to all do the second one and then remove this line.
android/app/build.gradle.kts
Outdated
| } | ||
|
|
||
| sentry { | ||
| org.set("emerge-tools") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the org need to be updated too? Actually what happens if you specify an org different from the auth token org?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
404, handled

Resolves EME-622