Push notification token management #5178

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.

Sign up for GitHub

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

Open

dgboss wants to merge 34 commits into main from task/fcm-tokens/4906

Open

Push notification token management #5178

Include device id

SonarQubeCloud / SonarCloud Code Analysis failed Mar 5, 2026 in 1m 30s

Quality Gate failed

Failed conditions
E Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check failure on line 60 in backend/packages/wps-api/src/app/routers/fcm.py

sonarqubecloud / SonarCloud Code Analysis

Document this HTTPException with status code 404 in the "responses" parameter.

See more on https://sonarcloud.io/project/issues?id=bcgov_wps&issues=AZy2cYEN47o5I3jkdUUe&open=AZy2cYEN47o5I3jkdUUe&pullRequest=5178

Check failure on line 83 in backend/packages/wps-api/src/app/tests/fcm/test_fcm_router.py

sonarqubecloud / SonarCloud Code Analysis

"token" detected here, make sure this is not a hard-coded secret.

See more on https://sonarcloud.io/project/issues?id=bcgov_wps&issues=AZy_biQywsr_rB1ek9NU&open=AZy_biQywsr_rB1ek9NU&pullRequest=5178

Check failure on line 96 in backend/packages/wps-api/src/app/tests/fcm/test_fcm_router.py

sonarqubecloud / SonarCloud Code Analysis

"token" detected here, make sure this is not a hard-coded secret.

See more on https://sonarcloud.io/project/issues?id=bcgov_wps&issues=AZy_biQywsr_rB1ek9NV&open=AZy_biQywsr_rB1ek9NV&pullRequest=5178

View more details on SonarQubeCloud