wordpress-mobile / WordPress-Android

WordPress for Android
http://android.wordpress.org
GNU General Public License v2.0
2.99k stars 1.33k forks source link

Enable Zendesk tickets from Automatticians #21453

Closed nbradbury closed 6 days ago

nbradbury commented 1 week ago

Fixes #21449

Automatticians who have a Zendesk staff member role (admin, agent, etc.) identity have long been unable to test creating Zendesk tickets in the apps.

According to Zendesk, a workaround for this is to programmatically append "+testing" when a ticket is submitted from an @automattic.com or @a8c.com email address. This PR does just that.

To test:

dangermattic commented 1 week ago
2 Warnings
:warning: strings.xml files should only be updated on release branches, when the translations are downloaded by our automation.
:warning: PR is not assigned to a milestone.

Generated by :no_entry_sign: Danger

wpmobilebot commented 1 week ago
WordPress📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress WordPress
FlavorJalapeno
Build TypeDebug
Versionpr21453-7d77222
Commit7d7722286e3edf1e4568fef280ef1a99008aa701
Direct Downloadwordpress-prototype-build-pr21453-7d77222.apk
Note: Google Login is not supported on these builds.
wpmobilebot commented 1 week ago
Jetpack📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack Jetpack
FlavorJalapeno
Build TypeDebug
Versionpr21453-7d77222
Commit7d7722286e3edf1e4568fef280ef1a99008aa701
Direct Downloadjetpack-prototype-build-pr21453-7d77222.apk
Note: Google Login is not supported on these builds.
codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 39.50%. Comparing base (535baed) to head (7d77222). Report is 70 commits behind head on trunk.

Files with missing lines Patch % Lines
.../java/org/wordpress/android/ui/prefs/AppPrefs.java 0.00% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## trunk #21453 +/- ## ======================================= Coverage 39.50% 39.50% ======================================= Files 2119 2119 Lines 99437 99437 Branches 15282 15282 ======================================= Hits 39281 39281 Misses 56674 56674 Partials 3482 3482 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

sonarcloud[bot] commented 1 week ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud