Skip to content

[Bug]: Notification Failing on Uniqueness Error (cardsubscription) #1171

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

Closed
StephanSkai opened this issue Jun 5, 2025 · 3 comments
Closed

Comments

@StephanSkai
Copy link

Where is the problem occurring?

I encountered the problem while interacting with the server (Backend)

What browsers are you seeing the problem on?

Chrome

Current behavior

Implemented telegram bot notifications. Successfully received test messages when connecting the telegram url, however on car change (move or comment), I see the following errors in the logs.

2025-06-02T04:41:27.233096811Z
2025-06-02T04:41:27.233148864Z Warning: Adapter sent back a uniqueness error, but that error references key(s) (card_id, user_id) which cannot
2025-06-02T04:41:27.233152861Z be matched up with the column names of any attributes in this model (⁠ cardsubscription ⁠).  This probably
2025-06-02T04:41:27.233155585Z means there is a bug in this adapter.
2025-06-02T04:41:27.233157701Z (Note for adapter implementors: If your adapter doesn't support granular reporting of the keys violated
2025-06-02T04:41:27.233159761Z in uniqueness errors, then just use an empty array for the ⁠ keys ⁠ property of this error.)
2025-06-02T04:41:27.233162077Z (Proceeding anyway as if these keys weren't included...)

And received no message regarding card change.

Desired behavior

In this situation, I'd expect a valid message to appear in my telegram chat notifying me of card change.

Steps to reproduce

I added my tgram service to notification panel. Tested it and successfully received the test message:

Test Title

This is a test markdown message!

and then tried to make a comment on a card I am subscribed to, or move a card, and failed to receive a message. Checked the logs and saw a bunch of the above logs.

Other information

No response

@StephanSkai
Copy link
Author

Image

@meltyshev
Copy link
Member

Hi! Thanks for reporting this.

First of all, this is just a warning. Waterline ORM can't handle uniqueness errors on multiple fields properly, so it can generally be ignored. However, the strange part is that we've already commented out this warning using this patch, so it shouldn't appear in the logs at all. This likely means the patch wasn't applied in your case, even though it should be applied automatically when you run npm install.

Regarding the notifications, have you tried triggering one from a different user? Notifications aren't sent to the user who performs the action. Also, for testing purposes, you can try setting the same Telegram notification URL in the board settings and then move a card. Since per-board notifications are triggered on every board action, you should receive one that way.

@StephanSkai
Copy link
Author

Fantastic. That worked.

Greatly appreciate your help @meltyshev.

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

No branches or pull requests

2 participants