[FR] Expose `link` / `link_android` in notifications for deep linking
Maintainer antworten meist innerhalb von 4 Tagen
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 45/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Ruhig
- Tech-Stack
- firebase, java
- Bereich
- api, backend-api-design
Rechercherichtung
Beginne damit, die Klassen Notification und AndroidNotification sowie die Zuordnung des HTTP API-Payloads zu finden. Prüfe, wie vorhandene Benachrichtigungseigenschaften serialisiert werden, und ermittle anschließend, wie link und link_android auf gcm.n.link und gcm.n.link_android abgebildet werden sollen. Die Aufgabe ist erledigt, wenn die offiziell unterstützten Felder die erforderlichen Deep-Link-Benachrichtigungs-Payloads erzeugen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Problem statement
Deep linking on Android requires the data field in the Intent to be set. As described in this SO post or firebase/firebase-android-sdk#6703, the only way to populate the the field in notifications created automatically by the Firebase Android SDK (when the app is in the background) is by setting the gcm.n.link or gcm.n.link_android keys in data, which have reserved prefixes and are therefore not supposed to be used (although setting them works, at least today).
Originally posted by @lehcar09 in #6703:
Alternatively, we can file a feature request to support the
linkorlink_androidfield. I’ll get back to you once I hear back from them.
So, this is the FR to officially support these :)
Strawman
A strawman proposal would be to add link properties to the Notification / AndroidNotification classes, which would then be mapped to gcm.n.link / gcm.n.link_android in the JSON payload. Equivalently, in the HTTP API instead of this:
{
"message": {
"notification": {
"title": "Lorem",
"body": "Ipsum"
},
"data": {
"gcm.n.link": "app://deep/link"
},
}
}
It would be this:
{
"message": {
"notification": {
"title": "Lorem",
"body": "Ipsum",
"link": "app://deep/link"
}
}
}
or this:
{
"message": {
"android": {
"notification": {
"title": "Lorem",
"body": "Ipsum",
"link": "app://deep/link"
}
}
}
}
Alternatives
The workaround suggested here of using setAction doesn't work, because actions needs to be declared in the Android manifest in order to match the intent, which conflicts with dynamically registering deep link targets.
Thanks!
- Vorherrschende Sprache
- Java
- Sterne
- 620
- Forks
- 306
- Ø Merge
- 4 T. 7 Std.
- Gemergte PRs (30 T.)
- 13
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus firebase/firebase-admin-java
-
google-http-client-jackson2 cannot be excluded even when a JsonFactory is configured via FirebaseOptionsEvtl. vergeben @lahirumaramba hat das vor 6 Tagen übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
firebase/firebase-admin-java#1231 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 4 Tagen
-
[BUG] RemoteConfig unit tests fail in non-English (e.g., Korean) Locales due to Date ParsingEvtl. vergeben @rathovarun1032 hat das vor 205 Tagen übernommen. Offenapi: remoteconfig
firebase/firebase-admin-java#1196 · 1 Kommentar · 2 zugewiesene Personen ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Auth list tenants is not pageableEvtl. vergeben @siddharthmohan619-eng hat das vor 210 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
firebase/firebase-admin-java#1192 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 4 Tagen
-
needs-triage type: feature request
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
firebase/firebase-admin-java#1149 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Opentelemetry-context conflict, please bump google-api-client dependencyEvtl. wieder frei @lahirumaramba hat das vor 336 Tagen übernommen, und es ist kein Pull Request offen. Offenapi: core
firebase/firebase-admin-java#1138 · 8 Kommentare · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 4 Tagen
Alle Issues in firebase/firebase-admin-java
Ähnliche Issues
-
[destination-snowflake] Custom domains rejected unlike source connectionsEvtl. vergeben @kuza55 hat das heute übernommen. Offenautoteam community connectors/destination/snowflake team/use
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
area-dashboard
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
component/operate kind/feature-request
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 1 Tag
-
Forge coverage prompts carry text the agent cannot act onEvtl. vergeben @graalvmbot hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
oracle/graalvm-reachability-metadata#10572 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Bundled migration plansOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
IBM/sample-app-mod#55 ·