Skip to content

Fix: social.dm safety warning#5

Merged
halp1 merged 3 commits into
halp1:mainfrom
visiblyisabelle:dm-warn-fix
May 17, 2026
Merged

Fix: social.dm safety warning#5
halp1 merged 3 commits into
halp1:mainfrom
visiblyisabelle:dm-warn-fix

Conversation

@visiblyisabelle
Copy link
Copy Markdown
Contributor

Fixes two issues:

  • The client now re-emits social.dm as unsafe__social.dm in accordance with the logged warning
  • The wrap call within Social#dm now listens for unsafe__social.dm instead of social.dm, suppressing the incorrect logged warning

@vercel
Copy link
Copy Markdown

vercel Bot commented May 17, 2026

@visiblyisabelle is attempting to deploy a commit to the Haelp Team on Vercel.

A member of the Team first needs to authorize it.

@halp1
Copy link
Copy Markdown
Owner

halp1 commented May 17, 2026

Fixed similarly but slightly differently in b49d408.

Removed handling for 'social.dm' case in core ribbon message processing.
@halp1 halp1 merged commit 11d825d into halp1:main May 17, 2026
1 of 2 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.

2 participants