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

RawMessageDeleteEvent documentation has different behavoir than the library itself #953

Closed
3 tasks done
SteffoSpieler opened this issue Dec 17, 2022 · 5 comments
Closed
3 tasks done
Labels
t: unconfirmed bug Type: bug - needs testing on if this is an issue

Comments

@SteffoSpieler
Copy link

Summary

Documentation has message_ids, Library has message_id

Reproduction Steps

  1. Create a Listener that is using on_raw_message_delete
  2. Import RawMessageDeleteEvent
  3. Try to use message_ids (that is shown in the docs)
  4. See warning as it doesn't exist

Minimal Reproducible Code

@Cog.listener()
async def on_raw_message_delete(self, payload: RawMessageDeleteEvent):
    for message_id in payload.message_ids:

Expected Results

I expect to see the same arguments in the library as in the documentation.

Actual Results

The documentation has message_ids and the library has message_id.

Intents

Intents(guilds=True, members=True, bans=False, emojis=True, integrations=False, webhooks=False, invites=False, voice_states=True, presences=False, messages=True, message_content=True, reactions=True, typing=False)

System Information

  • Python v3.9.13-final
  • nextcord v2.3.2-final
  • aiohttp v3.8.3
  • system info: Windows 10 10.0.22621

Checklist

  • I have searched the open issues for duplicates.
  • I have shown the entire traceback, if possible.
  • I have removed my token from display, if visible.

Additional Context

That's not directly a "bug", but it's imo the category it fits the best?

@SteffoSpieler SteffoSpieler added the t: unconfirmed bug Type: bug - needs testing on if this is an issue label Dec 17, 2022
@ooliver1
Copy link
Member

I don't think this is an issue. You are using RawMessageDeleteEvent - on_raw_message_delete, but are referencing the documentation to RawBulkMessageDeleteEvent - on_raw_bulk_message_delete, since bulk has multiple messages.

@SteffoSpieler
Copy link
Author

oh true. I missed that as when I clicked on the type RawMessageDeleteDelete, I got to https://docs.nextcord.dev/en/stable/api.html#nextcord.RawMessageDeleteEvent, which then shows the bulk version (without the title) for me. But only when clicking it. When pressing F5, I jump to the correct one 🤔
I use Firefox fyi

@ooliver1
Copy link
Member

Works just fine for me on Firefox aurora 109.0b3 (64-bit) when clicking the type from on_raw_message_delete.

@MaskDuck
Copy link
Contributor

MaskDuck commented Dec 18, 2022

can't repro either
also looked at Nextcord sources and see nothing wrong with it (master branch)

@SteffoSpieler
Copy link
Author

oh well... then I think it's just a weird bug with opening in a new tab as I only can reproduce it sometimes - with the same steps.
I'll close this issue now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t: unconfirmed bug Type: bug - needs testing on if this is an issue
Projects
No open projects
Status: Invalid/Closed
Development

No branches or pull requests

3 participants