-
Notifications
You must be signed in to change notification settings - Fork 185
fix: attempt to fix several issues related to the new cache #921
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
fix: attempt to fix several issues related to the new cache #921
Conversation
|
Have you tested it? |
The issue seems to occur randomly so I can't really test it/we don't know when it occurs |
Toricane
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
|
Now this Traceback (most recent call last):
File "C:\Users\User\OneDrive\python\discord bots\mangalib-bot\source\main.py", line 23, in <module>
client.start()
File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\interactions\client\bot.py", line 134, in start
raise e from e
File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\interactions\client\bot.py", line 132, in start
self._loop.run_until_complete(self._ready())
File "C:\Users\User\AppData\Local\Programs\Python\Python310\Lib\asyncio\base_events.py", line 646, in run_until_complete
return future.result()
File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\interactions\client\bot.py", line 391, in _ready
await self._login()
File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\interactions\client\bot.py", line 396, in _login
await self._websocket._establish_connection(self._shards, self._presence)
File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\interactions\api\gateway\client.py", line 196, in _establish_connection
await self._handle_connection(stream, shard, presence)
File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\interactions\api\gateway\client.py", line 264, in _handle_connection
self._dispatch_event(event, data)
File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\interactions\api\gateway\client.py", line 414, in _dispatch_event
before = model(**old_obj._json)
File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\interactions\api\models\attrs_utils.py", line 120, in __init__
super().__init__(**kwargs)
File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\interactions\api\models\attrs_utils.py", line 80, in __init__
self.__attrs_init__(**passed_kwargs) # type: ignore
File "<attrs generated init interactions.api.models.message.Message>", line 8, in __attrs_init__
File "C:\Users\User\AppData\Local\Programs\Python\Python310\lib\site-packages\attr\converters.py", line 43, in optional_converter
return converter(val)
TypeError: fromisoformat: argument must be str |
nonono LBTM |
|
@Damego merge or no merge? |
@EdVraz |
|
ok ill look |
# Conflicts: # interactions/client/get.py
Damego
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now LGTM!
Toricane
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
About
This pull request attempts to fixes this bug
Checklist
pre-committo format and lint the change(s) made.3.8.6and higher.