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

Support event relations on all post message functions #806

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

nvrWhere
Copy link
Collaborator

@nvrWhere nvrWhere commented Oct 4, 2024

Title

Quotient/room.h Outdated Show resolved Hide resolved
@KitsuneRal KitsuneRal added the enhancement A feature or change request for the library label Nov 18, 2024
…ecause we probably shouldn't allow someone to send a message with an arbritary type but not the correct content. This is replaced by postEmote and postNotice the two types that are probably valid from the original postMessage function.
@nvrWhere
Copy link
Collaborator Author

So I updated the functions to make them not invokable as discussed. Also since we're breaking API i messed with them a little.

I removed postMessage because we probably shouldn't allow someone to send a message with an arbitrary type but not the correct content. This is replaced by postEmote and postNotice which along with postHtmlText are the types that are probably valid from the original postMessage function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A feature or change request for the library
Projects
Status: In work
Development

Successfully merging this pull request may close these issues.

3 participants