-
Notifications
You must be signed in to change notification settings - Fork 79
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
Retrieving users for an event returns all users in guild from cache (Incorrectly) #622
Comments
Thanks for the report and sorry for the delay. I'll look into it when possible |
Back to you on this one, we don't seem to cache the members of the scheduled events regardless of the code provided being incorrect. |
The cache with MemberData items will still be filled when members are retrieved from the Guild. My code does that too. Therefore the cache will be filled and wrong items are returned. I'll make an attempt to fix this myself tomorrow. |
In order to fix this we need to introduce a new ScheduledEventMember that contains eventId - user - member given by discord |
Please keep an eye on #631 |
When I retrieve a GuildScheduledEvent and access it's users or members property. The list returned contains all users in the guild instead of all users that have signed up for the event. I am using caching and I traced this issue back to the following CacheEntitySupplier code:
At no point are users filtered for the relevant event. This information isn't stored in MemberData as far as I can tell.
The code that runs in the RestEntitySupplier does return the expected results of only users that have signed up for an event.
The text was updated successfully, but these errors were encountered: