Skip to content

Update cleanup#186

Merged
sdzhong merged 4 commits intomainfrom
stefanosiano/update-cleanup
Sep 22, 2025
Merged

Update cleanup#186
sdzhong merged 4 commits intomainfrom
stefanosiano/update-cleanup

Conversation

@stefanosiano
Copy link
Copy Markdown
Contributor

updated Sentry Android Gradle Plugin version to upgrade Android SDK version
Cleaned TODO items now unnecessary due to update

…ersion

updated necessary options
Enabled logs
Switched user feedback with new API
…ersion

Cleaned TODO items now unnecessary due to update
@stefanosiano
Copy link
Copy Markdown
Contributor Author

Let's merge when the new plugin version is out

@@ -181,43 +180,7 @@ private void launchUserFeedback(SentryId sentryId) {
options.setSubmitButtonLabel("Submit");
options.setCancelButtonLabel("Cancel");
options.setUseSentryUser(false);
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

by default we use the sentry user to fill the name and email of the feedback.
We should just decide what we want for the demo (i guess it makes sense to keep the sentry user)

Base automatically changed from stefanosiano/logs-user-feedback to main July 30, 2025 19:09
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 12, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (6d98989) to head (52e4e67).

Files with missing lines Patch % Lines
...ain/java/com/example/vu/android/MyApplication.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff          @@
##            main    #186   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files         16      16           
  Lines        899     885   -14     
  Branches      65      65           
=====================================
+ Misses       899     885   -14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stefanosiano stefanosiano marked this pull request as ready for review August 12, 2025 15:27
@stefanosiano stefanosiano requested a review from ndmanvar as a code owner August 12, 2025 15:27
@sdzhong
Copy link
Copy Markdown
Contributor

sdzhong commented Sep 22, 2025

Reviewed and noted that User Feedback change has been properly applied
Before
image

After with showUserFeedbackDialog
image

Reference error event which is linked to the User Feedback

@sdzhong sdzhong self-assigned this Sep 22, 2025
@sdzhong sdzhong self-requested a review September 22, 2025 20:45
@sdzhong sdzhong merged commit 77b1722 into main Sep 22, 2025
5 checks passed
@sdzhong sdzhong deleted the stefanosiano/update-cleanup branch September 22, 2025 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants