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

sendFollowUpMessage has incorrect in-line documentation #1229

Open
valzargaming opened this issue May 20, 2024 · 0 comments
Open

sendFollowUpMessage has incorrect in-line documentation #1229

valzargaming opened this issue May 20, 2024 · 0 comments

Comments

@valzargaming
Copy link
Member

valzargaming commented May 20, 2024

* @param bool $ephemeral Whether the created follow-up should be ephemeral. Will be ignored if the respond is previously ephemeral.

Per the official docs, flags can be set to 64 to mark the message as ephemeral, except when it is the first follow-up message to a deferred Interactions Response. In that case, the flags field will be ignored, and the ephemerality of the message will be determined by the flags value in your original ACK.

Per our in-line docs, ephemeral will be ignored if the respond is previously ephemeral.

Our docs are incorrect and should be adjusted to reflect the true nature of the request:

  • The first follow-up message will never be ephemeral.
  • All follow-up messages after the first will be ephemeral if the first follow-up message passed $ephemeral = true.
  • All follow-up messages after the first will respect their ephemeral flags only if the first follow-up message did not declare $ephemeral = true.
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

1 participant