Skip to content
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

Fix push notifications and revert variants renaming #414

Merged
merged 4 commits into from
Feb 8, 2022

Conversation

Florian14
Copy link
Contributor

@Florian14 Florian14 commented Feb 7, 2022

#402

Push notifications were broken for some devices since the renaming of the build variants in #375. There is no direct relation between the flavours name and the FCM token since the applicationId has not changed but reverting these changes seems to solve the problem.

Also, the google-services.json has been downloaded again from the google console and updated in the app. It contains now all the application variants (prod/preprod) in a single file which has been moved to the gplay root folder, instead of having one file per variant.

We'll see to rename the flavours in the future as this is not a priority change and does not affect the app behaviour.

@github-actions
Copy link

github-actions bot commented Feb 7, 2022

Unit Test Results

11 files  ±0  11 suites  ±0   7s ⏱️ -1s
83 tests ±0  83 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 9a1a7ce. ± Comparison against base commit 6c65c4e.

♻️ This comment has been updated with latest results.

Florian Renaud added 3 commits February 7, 2022 13:33
This reverts commit f4a0318.

Conflicts:
	vector/src/gplay/agent/google-services.json
This reverts commit cb788d4.

Conflicts:
	vector/src/gplay/preprod/google-services.json
@Florian14 Florian14 force-pushed the feature/fre/fix_fcm_token_error branch from b2efc3b to 25978d3 Compare February 7, 2022 12:48
@Florian14 Florian14 changed the title Revert variants renaming Fix push notifications and revert variants renaming Feb 7, 2022
@Florian14 Florian14 requested a review from Claire1817 February 7, 2022 12:57
@Florian14 Florian14 marked this pull request as ready for review February 7, 2022 12:57
@Florian14 Florian14 merged commit 74e27e9 into develop Feb 8, 2022
@Florian14 Florian14 deleted the feature/fre/fix_fcm_token_error branch February 8, 2022 14:21
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