[PR #61] Make Sentry completely optional #62

New Issue

2026-02-16T12:26:46-05:00

yindo commented

2026-02-16 12:26:46 -05:00

📋 Pull Request Information

Original PR: https://github.com/stoatchat/for-android/pull/61
Author: @Misofist
Created: 2/14/2026
Status: 🔄 Open

Base: dev ← Head: optional-sentry

📝 Commits (1)

fc2bce4 Make Sentry completely optional

📊 Changes

2 files changed (+23 additions, -12 deletions)

View changed files

📝 app/build.gradle.kts (+18 -9)
📝 app/src/main/java/chat/stoat/activities/MainActivity.kt (+5 -3)

📄 Description

This PR fully disables Sentry if the DSN option is left empty both in stoatbuild.properties and in the RVX_SENTRY_DSN environment variable.

Not only does it prevent Sentry from being Init'd at runtime, but it also disables the Sentry Gradle plugin, since otherwise the plugin throws errors during Gradle syncs.

Sentry is only really necessary if you're shipping an app, not just developing locally. Making Sentry optional should reduce the barrier for entry to new devs.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/stoatchat/for-android/pull/61 **Author:** [@Misofist](https://github.com/Misofist) **Created:** 2/14/2026 **Status:** 🔄 Open **Base:** `dev` ← **Head:** `optional-sentry` --- ### 📝 Commits (1) - [`fc2bce4`](https://github.com/stoatchat/for-android/commit/fc2bce4541e8158ab3afd3a2b35880c152ee8c61) Make Sentry completely optional ### 📊 Changes **2 files changed** (+23 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `app/build.gradle.kts` (+18 -9) 📝 `app/src/main/java/chat/stoat/activities/MainActivity.kt` (+5 -3) </details> ### 📄 Description This PR fully disables Sentry if the DSN option is left empty both in stoatbuild.properties and in the RVX_SENTRY_DSN environment variable. Not only does it prevent Sentry from being Init'd at runtime, but it also disables the Sentry Gradle plugin, since otherwise the plugin throws errors during Gradle syncs. Sentry is only really necessary if you're shipping an app, not just developing locally. Making Sentry optional should reduce the barrier for entry to new devs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

yindo added the pull-request label 2026-02-16 12:26:46 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: stoatchat/for-android#62