Skip to content

Add content_private_body notification field#889

Merged
magopian merged 4 commits into
mainfrom
875-add-content_private_body-notification-field
May 21, 2026
Merged

Add content_private_body notification field#889
magopian merged 4 commits into
mainfrom
875-add-content_private_body-notification-field

Conversation

@magopian
Copy link
Copy Markdown
Collaborator

Fixes #875

Copy link
Copy Markdown
Collaborator

@cdesquilbet-dila cdesquilbet-dila left a comment

Choose a reason for hiding this comment

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

Est-ce qu'il y a un dév particulier à faire pour ne pas que content_private_body soit passé à Firebase ?

@magopian
Copy link
Copy Markdown
Collaborator Author

Est-ce qu'il y a un dév particulier à faire pour ne pas que content_private_body soit passé à Firebase ?

Les notifications push (envoyées à Firebase) sont créées et envoyées par le fichier push.py.

Les données envoyées sont construites ici, et on peut voir qu'il n'y a pas le content_private_body.

C'est ensuite vérifié dans les tests à cet endroit

@cdesquilbet-dila
Copy link
Copy Markdown
Collaborator

Parfait, merci !

@magopian magopian force-pushed the 875-add-content_private_body-notification-field branch from 24a6b77 to b8e1bc9 Compare May 21, 2026 12:12
@magopian magopian merged commit a82db61 into main May 21, 2026
5 checks passed
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.

Ajouter un attribut content_private_body aux notifications

2 participants