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

[bug] Client created with same client ID or device token but different tenant ID will break association with notifications #338

Open
chris13524 opened this issue May 21, 2024 · 6 comments
Labels
accepted The issue has been accepted into the project

Comments

@chris13524
Copy link
Member

Client created with same client ID or device token, but under different tenant will update the tenant ID of the client but not delete any associated notifications with the old tenant ID. Notifications are always queried by their tenant ID vs clients are not.

Two solutions:

  • Remove tenant_id column from notifications and use the tenant_id column of the clients table (if this is even needed at all)
  • Update the tenant_id or delete notifications when the client's tenant_id changes
@arein arein added the accepted The issue has been accepted into the project label May 21, 2024
@chris13524
Copy link
Member Author

@jdorman16

This comment was marked as spam.

@jdorman16

This comment was marked as spam.

@jdorman16

This comment was marked as spam.

@jdorman16

This comment was marked as spam.

@jdorman16

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted The issue has been accepted into the project
Projects
None yet
Development

No branches or pull requests

3 participants