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

direct messages aren't working #515

Open
sanslash332 opened this issue Apr 7, 2019 · 5 comments

Comments

Projects
None yet
5 participants
@sanslash332
Copy link

commented Apr 7, 2019

Hello,

I'm using twython 3.7.0 (indirectly) through a twitter client named "theqube"

WhenI'm trying to use any of the direct messages features, throws a twython exception, because the twitter sends a 404 not found because twithon is trying to connect to wrong api.

Exactly, the exception is this :

context; simple update command for get new direct messages.

TwythonError: Twitter API returned a 404 (Not Found), Sorry, that page does not exist.
Sun Apr 07, 2019 11:28:59 core.buffers.updating ERROR: tfj: Error during update process in buffer buffer Direct Messages
Traceback (most recent call last):
File "D:\Programacion\python\TheQube\src\core\sessions\buffers\buffers\updating.py", line 87, in update
new = self.retrieve_update(update_type=update_type)
File "D:\Programacion\python\TheQube\src\session\twitter\buffers\directs.py", line 25, in retrieve_update
return self.timeline_update(update_function_name=’get_direct_messages’, since_id=self.get_max_twitter_id(), include_entities=True, full_text=True, tweet_mode=’extended’)
File "D:\Programacion\python\TheQube\src\session\twitter\buffers\api_count.py", line 49, in timeline_update
new_data = self.session.api_call(update_function_name, report_success=False, report_failure=False, *args, **kwargs)
File "D:\Programacion\python\TheQube\src\session\twitter\twitter.py", line 415, in api_call
raise e
TwythonError: Twitter API returned a 404 (Not Found), Sorry, that page does not exist.

please tell me if the lib is outdated, or the client needs to update itself to use the new features of the twython lib.

%Thanks

@Menelion

This comment has been minimized.

Copy link
Contributor

commented Apr 9, 2019

Hey @sanslash332, theQube maintainer here :). First of all, thanks for using the client! Basically, I don't have direct messages myself. I'm kind of against changing the library from Twython to something else, and also against embedding a patched version of Twython into my code. So, until #500 is merged, no direct messages for all of us :(.

@ryanmcgrath

This comment has been minimized.

Copy link
Owner

commented Apr 9, 2019

@sanslash332

This comment has been minimized.

Copy link
Author

commented Apr 9, 2019

so @Menelion why not use directly the @manuelcortez fork into theqube instead this official package that is missing the #500 pr?

So, that Pr have more than 6 months waiting for an approval u.u

@mikehelmick

This comment has been minimized.

Copy link

commented Apr 10, 2019

Tagging @michaelhelmick instead.

@michaelhelmick

This comment has been minimized.

Copy link
Collaborator

commented Apr 10, 2019

#500 is merged, I'll try and get a release out this week!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.