Skip to content

Support fcm for user pins#17

Draft
jibraniqbal666 wants to merge 7 commits intopebble-dev:masterfrom
jibraniqbal666:feature/support-fcm
Draft

Support fcm for user pins#17
jibraniqbal666 wants to merge 7 commits intopebble-dev:masterfrom
jibraniqbal666:feature/support-fcm

Conversation

@jibraniqbal666
Copy link
Copy Markdown

@jibraniqbal666 jibraniqbal666 commented Mar 8, 2026

This support registering and sending fcm to core app, this only supports user pins.
Here is the mobile side of implementation coredevices/mobileapp#131

we need to update submodule of rws_common with

from setuptools import setup, find_packages

setup(
    name='rws-common',
    version='0.1',
    packages=find_packages(),
    platforms='any',
    install_requires=[
        'Flask',
        'honeycomb-beeline==2.11.4'
    ]
)

The update to python 3.9 is require due to firebase 7.2.0 have the functions such as 'send_each_for_multicast' which is required.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We don't need this change, you should just drop it

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also drop this

init_app(app)
init_api(app) # Includes both private (timeline-sync) and public (timeline-api) APIs

cred = credentials.Certificate('./google-services.json')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This won't work, we need a config as env, not as a file, since the filesystem on the deployed workers always changes

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

can you check how it can be achieve?
It can see that we can inject json as env as well, but I am only seeing this using a json file either for services.json or service-account.json

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There are a couple ways, some mentioned in https://firebase.google.com/docs/admin/setup#initialize_the_sdk

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