Skip to content

Clean up documentation. #5155

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

Merged
merged 1 commit into from
Jul 11, 2023
Merged

Clean up documentation. #5155

merged 1 commit into from
Jul 11, 2023

Conversation

gsakakihara
Copy link
Contributor

  • Removed deprecated class documentation about upstream messaging.
  • Updated send deprecation wording to "FCM upstream messaging" to align with the standard naming.
  • Fixed spelling error.

* Removed deprecated class documentation about upstream messaging.
* Updated send deprecation wording to "FCM upstream messaging" to align with the standard naming.
* Fixed spelling error.
@github-actions
Copy link
Contributor

github-actions bot commented Jul 11, 2023

Javadoc Changes:
--- /Users/runner/diff/original/firebase-kotlindoc/android/com/google/firebase/messaging/FirebaseMessaging.html	2023-07-11 16:59:21.000000000 +0000
+++ /Users/runner/diff/modified/firebase-kotlindoc/android/com/google/firebase/messaging/FirebaseMessaging.html	2023-07-11 16:54:12.000000000 +0000
@@ -13,12 +13,6 @@
     <hr>
     <p>Top level <a href="https://firebase.google.com/docs/cloud-messaging/">Firebase Cloud Messaging</a> singleton that provides methods for subscribing to topics and sending upstream messages. </p>
     <p>In order to receive Firebase messages, declare an implementation of <code><a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessagingService.html">FirebaseMessagingService</a></code> in the app manifest. To process messages, override base class methods to handle any events required by the application. </p>
-    <p>Client apps can send <a href="https://firebase.google.com/docs/cloud-messaging/android/upstream">upstream messages</a> back to the app server using the XMPP-based Cloud Connection Server. For example: </p>
-    <pre class="prettyprint">FirebaseMessaging.getInstance().send(
-    new RemoteMessage.Builder(SENDER_ID + &quot;@fcm.googleapis.com&quot;)
-    .setMessageId(id)
-    .addData(&quot;key&quot;, &quot;value&quot;)
-.build());</pre>
     <h2>Summary</h2>
     <div class="devsite-table-wrapper">
       <table class="responsive">
@@ -102,7 +96,7 @@
             <td>
               <div><code><span><del><a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessaging.html#send(com.google.firebase.messaging.RemoteMessage)">send</a></del></span>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/messaging/RemoteMessage.html">RemoteMessage</a>&nbsp;message)</code></div>
               <p><strong>This method is deprecated.</strong>
-                <p>FCM upstream is deprecated and will be decommissioned in June 2024.</p>
+                <p>FCM upstream messaging is deprecated and will be decommissioned in June 2024.</p>
               </p>
             </td>
           </tr>
@@ -278,7 +272,7 @@
         <h3 class="api-name" id="send(com.google.firebase.messaging.RemoteMessage)">send</h3>
         <pre class="api-signature no-pretty-print">public&nbsp;void&nbsp;<span><del><a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessaging.html#send(com.google.firebase.messaging.RemoteMessage)">send</a></del></span>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="/docs/reference/android/com/google/firebase/messaging/RemoteMessage.html">RemoteMessage</a>&nbsp;message)</pre>
         <aside class="caution"><strong>This method is deprecated.</strong><br>
-          <p>FCM upstream is deprecated and will be decommissioned in June 2024. Learn more in the <a href="https://firebase.google.com/support/faq#fcm-23-deprecation">FAQ about FCM features deprecated in June 2023</a>.</p>
+          <p>FCM upstream messaging is deprecated and will be decommissioned in June 2024. Learn more in the <a href="https://firebase.google.com/support/faq#fcm-23-deprecation">FAQ about FCM features deprecated in June 2023</a>.</p>
         </aside>
         <p>Sends <code>message</code> upstream to your app server. </p>
         <p>When there is an active connection the message will be sent immediately, otherwise the message will be queued up to the time to live (TTL) set in the message.</p>
@@ -405,7 +399,7 @@
         <pre class="api-signature no-pretty-print">public&nbsp;@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developers.google.com/android/reference/com/google/android/gms/tasks/Task.html">Task</a>&lt;<a href="https://developer.android.com/reference/kotlin/java/lang/Void.html">Void</a>&gt;&nbsp;<a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessaging.html#subscribeToTopic(java.lang.String)">subscribeToTopic</a>(@<a href="https://developer.android.com/reference/kotlin/androidx/annotation/NonNull.html">NonNull</a> <a href="https://developer.android.com/reference/kotlin/java/lang/String.html">String</a>&nbsp;topic)</pre>
         <p>Subscribes to <code>topic</code> in the background. </p>
         <p>The subscribe operation is persisted and will be retried until successful. </p>
-        <p>This uses an FCM registration token to identify the app instance, generating one if it does not exist (see <code><a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessaging.html#getToken()">getToken</a></code>), which perodically sends data to the Firebase backend when auto-init is enabled. To delete the data, delete the token (<code><a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessaging.html#deleteToken()">deleteToken</a></code>) and the Firebase Installations ID (<code>FirebaseInstallations.delete()</code>). To stop the periodic sending of data, disable auto-init (<code><a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessaging.html#setAutoInitEnabled(boolean)">setAutoInitEnabled</a></code>).</p>
+        <p>This uses an FCM registration token to identify the app instance, generating one if it does not exist (see <code><a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessaging.html#getToken()">getToken</a></code>), which periodically sends data to the Firebase backend when auto-init is enabled. To delete the data, delete the token (<code><a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessaging.html#deleteToken()">deleteToken</a></code>) and the Firebase Installations ID (<code>FirebaseInstallations.delete()</code>). To stop the periodic sending of data, disable auto-init (<code><a href="/docs/reference/android/com/google/firebase/messaging/FirebaseMessaging.html#setAutoInitEnabled(boolean)">setAutoInitEnabled</a></code>).</p>
         <div class="devsite-table-wrapper">
           <table class="responsive">
             <colgroup>
--- /Users/runner/diff/original/firebase-kotlindoc/kotlin/com/google/firebase/messaging/FirebaseMessaging.html	2023-07-11 16:59:21.000000000 +0000
+++ /Users/runner/diff/modified/firebase-kotlindoc/kotlin/com/google/firebase/messaging/FirebaseMessaging.html	2023-07-11 16:54:12.000000000 +0000
@@ -13,12 +13,6 @@
     <hr>
     <p>Top level <a href="https://firebase.google.com/docs/cloud-messaging/">Firebase Cloud Messaging</a> singleton that provides methods for subscribing to topics and sending upstream messages. </p>
     <p>In order to receive Firebase messages, declare an implementation of <code><a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessagingService.html">FirebaseMessagingService</a></code> in the app manifest. To process messages, override base class methods to handle any events required by the application. </p>
-    <p>Client apps can send <a href="https://firebase.google.com/docs/cloud-messaging/android/upstream">upstream messages</a> back to the app server using the XMPP-based Cloud Connection Server. For example: </p>
-    <pre class="prettyprint">FirebaseMessaging.getInstance().send(
-    new RemoteMessage.Builder(SENDER_ID + &quot;@fcm.googleapis.com&quot;)
-    .setMessageId(id)
-    .addData(&quot;key&quot;, &quot;value&quot;)
-.build());</pre>
     <h2>Summary</h2>
     <div class="devsite-table-wrapper">
       <table class="responsive">
@@ -102,7 +96,7 @@
             <td>
               <div><code><span><del><a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessaging.html#send(com.google.firebase.messaging.RemoteMessage)">send</a></del></span>(message:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/messaging/RemoteMessage.html">RemoteMessage</a>)</code></div>
               <p><strong>This function is deprecated.</strong>
-                <p>FCM upstream is deprecated and will be decommissioned in June 2024.</p>
+                <p>FCM upstream messaging is deprecated and will be decommissioned in June 2024.</p>
               </p>
             </td>
           </tr>
@@ -278,7 +272,7 @@
         <h3 class="api-name" id="send(com.google.firebase.messaging.RemoteMessage)">send</h3>
         <pre class="api-signature no-pretty-print">fun&nbsp;<span><del><a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessaging.html#send(com.google.firebase.messaging.RemoteMessage)">send</a></del></span>(message:&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/messaging/RemoteMessage.html">RemoteMessage</a>):&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-unit/index.html">Unit</a></pre>
         <aside class="caution"><strong>This function is deprecated.</strong><br>
-          <p>FCM upstream is deprecated and will be decommissioned in June 2024. Learn more in the <a href="https://firebase.google.com/support/faq#fcm-23-deprecation">FAQ about FCM features deprecated in June 2023</a>.</p>
+          <p>FCM upstream messaging is deprecated and will be decommissioned in June 2024. Learn more in the <a href="https://firebase.google.com/support/faq#fcm-23-deprecation">FAQ about FCM features deprecated in June 2023</a>.</p>
         </aside>
         <p>Sends <code>message</code> upstream to your app server. </p>
         <p>When there is an active connection the message will be sent immediately, otherwise the message will be queued up to the time to live (TTL) set in the message.</p>
@@ -405,7 +399,7 @@
         <pre class="api-signature no-pretty-print">fun&nbsp;<a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessaging.html#subscribeToTopic(java.lang.String)">subscribeToTopic</a>(topic:&nbsp;<a href="https://kotlinlang.org/api/latest/jvm/stdlib/kotlin/-string/index.html">String</a>):&nbsp;<a href="https://developers.google.com/android/reference/com/google/android/gms/tasks/Task.html">Task</a>&lt;<a href="https://developer.android.com/reference/kotlin/java/lang/Void.html">Void</a>!&gt;</pre>
         <p>Subscribes to <code>topic</code> in the background. </p>
         <p>The subscribe operation is persisted and will be retried until successful. </p>
-        <p>This uses an FCM registration token to identify the app instance, generating one if it does not exist (see <code><a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessaging.html#getToken()">getToken</a></code>), which perodically sends data to the Firebase backend when auto-init is enabled. To delete the data, delete the token (<code><a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessaging.html#deleteToken()">deleteToken</a></code>) and the Firebase Installations ID (<code>FirebaseInstallations.delete()</code>). To stop the periodic sending of data, disable auto-init (<code><a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessaging.html#setAutoInitEnabled(boolean)">setAutoInitEnabled</a></code>).</p>
+        <p>This uses an FCM registration token to identify the app instance, generating one if it does not exist (see <code><a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessaging.html#getToken()">getToken</a></code>), which periodically sends data to the Firebase backend when auto-init is enabled. To delete the data, delete the token (<code><a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessaging.html#deleteToken()">deleteToken</a></code>) and the Firebase Installations ID (<code>FirebaseInstallations.delete()</code>). To stop the periodic sending of data, disable auto-init (<code><a href="/docs/reference/kotlin/com/google/firebase/messaging/FirebaseMessaging.html#setAutoInitEnabled(boolean)">setAutoInitEnabled</a></code>).</p>
         <div class="devsite-table-wrapper">
           <table class="responsive">
             <colgroup>

@google-oss-bot
Copy link
Contributor

Coverage Report 1

Affected Products

  • firebase-messaging

    Overall coverage changed from 84.14% (8fff029) to 84.25% (840602b) by +0.11%.

    FilenameBase (8fff029)Merge (840602b)Diff
    FirebaseMessaging.java73.57%74.45%+0.88%
    MessagingAnalytics.java81.78%81.38%-0.40%
    RequestDeduplicator.java80.00%90.00%+10.00%

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/pDyL3K1N3K.html

@github-actions
Copy link
Contributor

Unit Test Results

  54 files  ±0    54 suites  ±0   7m 58s ⏱️ +53s
470 tests ±0  470 ✔️ ±0  0 💤 ±0  0 ±0 
940 runs  ±0  940 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit a49c6bf. ± Comparison against base commit 8fff029.

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • base

    TypeBase (8fff029)Merge (840602b)Diff
    apk (aggressive)?8.39 kB? (?)
    apk (release)?8.66 kB? (?)
  • firebase-messaging

    TypeBase (8fff029)Merge (840602b)Diff
    aar?142 kB? (?)
    apk (aggressive)?465 kB? (?)
    apk (release)?1.72 MB? (?)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/Rb9lklRTwi.html

@google-oss-bot
Copy link
Contributor

Startup Time Report 1

Note: Layout is sometimes suboptimal due to limited formatting support on GitHub. Please check this report on GCS.

Notes

Startup Times

  • fire-fcm

    DeviceStatisticsDistributions
    oriole-32
    Percentile8fff029840602bDiffSignificant (?)
    p101.34 ±1 ms929 ±1002 μs-411 μs (-30.7%)NO
    p251.43 ±1 ms995 ±1100 μs-438 μs (-30.5%)NO
    p501.61 ±2 ms1.10 ±1 ms-507 μs (-31.5%)NO
    p751.85 ±2 ms1.31 ±1 ms-538 μs (-29.1%)NO
    p902.24 ±2 ms1.62 ±2 ms-615 μs (-27.5%)NO

    20 test runs in comparison
    CommitTest Runs
    8fff029
    • 2023-07-10_18:32:46.098718_dVvA
    • 2023-07-10_18:32:46.102914_NfwX
    • 2023-07-10_18:32:46.102931_ampP
    • 2023-07-10_18:32:46.102938_ysjh
    • 2023-07-10_18:32:46.102943_Aqzy
    • 2023-07-10_18:32:46.102949_Ugrw
    • 2023-07-10_18:32:46.102954_RoSY
    • 2023-07-10_18:32:46.102960_nach
    • 2023-07-10_18:32:46.102965_pSIX
    • 2023-07-10_18:32:46.102973_KTBx
    840602b
    • 2023-07-11_16:58:31.159562_ypzr
    • 2023-07-11_16:58:31.166123_fmBx
    • 2023-07-11_16:58:31.166139_IZMW
    • 2023-07-11_16:58:31.166148_xGXT
    • 2023-07-11_16:58:31.166154_YFQd
    • 2023-07-11_16:58:31.166161_hWIA
    • 2023-07-11_16:58:31.166173_ohIg
    • 2023-07-11_16:58:31.166179_sUnk
    • 2023-07-11_16:58:31.166185_XKDb
    • 2023-07-11_16:58:31.166192_FCbi
    redfin-30
    Percentile8fff029840602bDiffSignificant (?)
    p101.37 ±0.7 ms3.26 ±4 ms+1.89 ms (+138.7%)NO
    p251.47 ±0.8 ms3.53 ±4 ms+2.06 ms (+140.2%)NO
    p501.66 ±1 ms3.96 ±5 ms+2.30 ms (+138.8%)NO
    p752.01 ±1 ms4.58 ±5 ms+2.57 ms (+127.7%)NO
    p902.57 ±2 ms5.60 ±6 ms+3.03 ms (+117.6%)NO

    20 test runs in comparison
    CommitTest Runs
    8fff029
    • 2023-07-10_18:32:46.098718_dVvA
    • 2023-07-10_18:32:46.102914_NfwX
    • 2023-07-10_18:32:46.102931_ampP
    • 2023-07-10_18:32:46.102938_ysjh
    • 2023-07-10_18:32:46.102943_Aqzy
    • 2023-07-10_18:32:46.102949_Ugrw
    • 2023-07-10_18:32:46.102954_RoSY
    • 2023-07-10_18:32:46.102960_nach
    • 2023-07-10_18:32:46.102965_pSIX
    • 2023-07-10_18:32:46.102973_KTBx
    840602b
    • 2023-07-11_16:58:31.159562_ypzr
    • 2023-07-11_16:58:31.166123_fmBx
    • 2023-07-11_16:58:31.166139_IZMW
    • 2023-07-11_16:58:31.166148_xGXT
    • 2023-07-11_16:58:31.166154_YFQd
    • 2023-07-11_16:58:31.166161_hWIA
    • 2023-07-11_16:58:31.166173_ohIg
    • 2023-07-11_16:58:31.166179_sUnk
    • 2023-07-11_16:58:31.166185_XKDb
    • 2023-07-11_16:58:31.166192_FCbi
  • timeToInitialDisplay

    DeviceStatisticsDistributions
    oriole-32
    Percentile8fff029840602bDiffSignificant (?)
    p10200 ±4 ms204 ±1 ms+4.56 ms (+2.3%)NO
    p25206 ±5 ms210 ±2 ms+4.38 ms (+2.1%)NO
    p50214 ±5 ms218 ±3 ms+4.48 ms (+2.1%)NO
    p75222 ±6 ms229 ±7 ms+6.63 ms (+3.0%)NO
    p90231 ±6 ms244 ±19 ms+13.7 ms (+5.9%)NO

    20 test runs in comparison
    CommitTest Runs
    8fff029
    • 2023-07-10_18:32:46.098718_dVvA
    • 2023-07-10_18:32:46.102914_NfwX
    • 2023-07-10_18:32:46.102931_ampP
    • 2023-07-10_18:32:46.102938_ysjh
    • 2023-07-10_18:32:46.102943_Aqzy
    • 2023-07-10_18:32:46.102949_Ugrw
    • 2023-07-10_18:32:46.102954_RoSY
    • 2023-07-10_18:32:46.102960_nach
    • 2023-07-10_18:32:46.102965_pSIX
    • 2023-07-10_18:32:46.102973_KTBx
    840602b
    • 2023-07-11_16:58:31.159562_ypzr
    • 2023-07-11_16:58:31.166123_fmBx
    • 2023-07-11_16:58:31.166139_IZMW
    • 2023-07-11_16:58:31.166148_xGXT
    • 2023-07-11_16:58:31.166154_YFQd
    • 2023-07-11_16:58:31.166161_hWIA
    • 2023-07-11_16:58:31.166173_ohIg
    • 2023-07-11_16:58:31.166179_sUnk
    • 2023-07-11_16:58:31.166185_XKDb
    • 2023-07-11_16:58:31.166192_FCbi
    redfin-30
    Percentile8fff029840602bDiffSignificant (?)
    p10245 ±5 ms266 ±4 ms+20.7 ms (+8.4%)MAYBE
    p25251 ±6 ms272 ±4 ms+20.8 ms (+8.3%)MAYBE
    p50259 ±5 ms280 ±4 ms+21.2 ms (+8.2%)MAYBE
    p75267 ±5 ms289 ±4 ms+22.0 ms (+8.2%)MAYBE
    p90277 ±6 ms302 ±5 ms+25.3 ms (+9.1%)MAYBE

    20 test runs in comparison
    CommitTest Runs
    8fff029
    • 2023-07-10_18:32:46.098718_dVvA
    • 2023-07-10_18:32:46.102914_NfwX
    • 2023-07-10_18:32:46.102931_ampP
    • 2023-07-10_18:32:46.102938_ysjh
    • 2023-07-10_18:32:46.102943_Aqzy
    • 2023-07-10_18:32:46.102949_Ugrw
    • 2023-07-10_18:32:46.102954_RoSY
    • 2023-07-10_18:32:46.102960_nach
    • 2023-07-10_18:32:46.102965_pSIX
    • 2023-07-10_18:32:46.102973_KTBx
    840602b
    • 2023-07-11_16:58:31.159562_ypzr
    • 2023-07-11_16:58:31.166123_fmBx
    • 2023-07-11_16:58:31.166139_IZMW
    • 2023-07-11_16:58:31.166148_xGXT
    • 2023-07-11_16:58:31.166154_YFQd
    • 2023-07-11_16:58:31.166161_hWIA
    • 2023-07-11_16:58:31.166173_ohIg
    • 2023-07-11_16:58:31.166179_sUnk
    • 2023-07-11_16:58:31.166185_XKDb
    • 2023-07-11_16:58:31.166192_FCbi

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/7ROJLPBnMX/index.html

@gsakakihara gsakakihara merged commit 4d1cc39 into master Jul 11, 2023
@gsakakihara gsakakihara deleted the gsakakihara/documentation branch July 11, 2023 20:22
Copy link
Contributor

@egilmorez egilmorez left a comment

Choose a reason for hiding this comment

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

LG, thanks!

@firebase firebase locked and limited conversation to collaborators Aug 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants