Skip to content

Conversation

@VincentRPS
Copy link

definitely doesn't work yet

definitely doesn't work yet, just felt like there was way too much changes not to commit already
@VincentRPS VincentRPS requested a review from Paillat-dev May 15, 2025 19:54
Copy link
Member

@Paillat-dev Paillat-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't try running the code yet, will do that later today

Co-authored-by: Paillat <[email protected]>
Signed-off-by: VincentRPS <[email protected]>
@Paillat-dev
Copy link
Member

@VincentRPS You should also import defaultdict

@Paillat-dev
Copy link
Member

Any news ? See traceback of another error here:

https://canary.discord.com/channels/881207955029110855/881735314987708456/1373639761696133160

@Paillat-dev
Copy link
Member

Congrats @VincentRPS you successfully broke the docs ! 🎉

@Paillat-dev
Copy link
Member

Paillat-dev commented Jul 29, 2025

@VincentRPS I am going to rebase this for you and push here

@VincentRPS
Copy link
Author

@VincentRPS I am going to rebase this for you and push here

M'kay!

@Paillat-dev Paillat-dev changed the title refactor: ConnectionState & Event Handling refactor!: ConnectionState & Event Handling Sep 5, 2025
@Paillat-dev Paillat-dev linked an issue Sep 8, 2025 that may be closed by this pull request
)
return

member = await guild.get_member(int(data["user"]["id"]))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Normally people aren't members of guilds that they're being unbanned from

Copy link

@EmmmaTech EmmmaTech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:3

return await self.cache.get_all_stickers()

async def get_emoji(self, emoji_id: int | None) -> GuildEmoji | AppEmoji | None:
return await self.get_emoji(emoji_id)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
return await self.get_emoji(emoji_id)
return await self.cache.get_emoji(emoji_id)

await state.maybe_store_app_emoji(state.application_id, e)


class GuildCreate(Event, Guild):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this (among a few other events) appears to be duplicated in guild.py?

else:
self.dispatch("guild_join", guild)

def parse_guild_audit_log_entry_create(self, data) -> None:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume these leftover parse functions have yet to be converted into events?

@Lumabots
Copy link

@VincentRPS are you still working on this ?

@Lumabots Lumabots closed this Oct 24, 2025
@Lumabots Lumabots reopened this Oct 24, 2025
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.

Rewrite a better caching system

6 participants