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

docs(Message): improve message snapshots description #10709

Merged
merged 3 commits into from
Jan 24, 2025

Conversation

Amgelo563
Copy link
Contributor

@Amgelo563 Amgelo563 commented Jan 18, 2025

Please describe the changes this PR makes and why it should be merged:

This improves the Message#messageSnapshots documentation, since the current one doesn't mention the snapshots themselves. Mainly opening the PR to gather comments on it if necessary.

Status and versioning classification:

  • This PR only includes non-code changes, like changes to documentation, README, etc.

@Amgelo563 Amgelo563 requested a review from a team as a code owner January 18, 2025 07:08
Copy link

vercel bot commented Jan 18, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
discord-js ⬜️ Ignored (Inspect) Visit Preview Jan 24, 2025 10:28am
discord-js-guide ⬜️ Ignored (Inspect) Visit Preview Jan 24, 2025 10:28am

packages/discord.js/src/structures/Message.js Outdated Show resolved Hide resolved
@Amgelo563 Amgelo563 requested a review from Jiralite January 21, 2025 16:51
@kodiakhq kodiakhq bot merged commit 54d8750 into discordjs:main Jan 24, 2025
7 checks passed
Jiralite added a commit that referenced this pull request Jan 25, 2025
* docs(Message): improve message snapshots description

* docs(Message): remove snapshots single entry callout

---------

Co-authored-by: Jiralite <33201955+Jiralite@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants