Skip to content

fix: [SDK-4946] register with Firebase Installation ID when the legacy FCM token API is disabled - Revert - #2719

Merged
abdulraqeeb33 merged 1 commit into
mainfrom
revert-2700-fadi/SDK-4946-fcm-fid-registration
Aug 19, 2026
Merged

fix: [SDK-4946] register with Firebase Installation ID when the legacy FCM token API is disabled - Revert#2719
abdulraqeeb33 merged 1 commit into
mainfrom
revert-2700-fadi/SDK-4946-fcm-fid-registration

Conversation

@abdulraqeeb33

Copy link
Copy Markdown
Contributor

Reverts #2700

@abdulraqeeb33
abdulraqeeb33 requested a review from a team as a code owner August 19, 2026 20:28
@linear-code

linear-code Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

@abdulraqeeb33 abdulraqeeb33 changed the title Revert "fix: [SDK-4946] register with Firebase Installation ID when the legacy FCM token API is disabled" fix: [SDK-4946] register with Firebase Installation ID when the legacy FCM token API is disabled - Revert Aug 19, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📊 Diff Coverage Report

Diff Coverage Report (Changed Lines Only)

Gate: aggregate coverage on changed executable lines must be ≥ 80% (JaCoCo line data for lines touched in the diff).

Changed Files Coverage

  • PushRegistratorFCM.kt: 0/6 touched executable lines (0.0%) (9 touched lines in diff)
    • 6 uncovered touched lines in this file

Overall (aggregate gate)

0/6 touched executable lines covered (0.0% — requires ≥ 80%)

Per-file detail (informational; gate is aggregate above):

  • PushRegistratorFCM.kt: 0.0% (6 uncovered touched lines)

❌ Coverage Check Failed

Aggregate coverage on touched lines is 0.0% (minimum 80%).

📥 View workflow run

@abdulraqeeb33
abdulraqeeb33 merged commit 98e64f5 into main Aug 19, 2026
6 of 9 checks passed
@abdulraqeeb33
abdulraqeeb33 deleted the revert-2700-fadi/SDK-4946-fcm-fid-registration branch August 19, 2026 20:34
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