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

Use API to get_user_str_from_profile #325

Closed
wants to merge 1 commit into from

Conversation

electricfish1
Copy link

fixes #320

@datawhores
Copy link
Collaborator

@jbsparrow is working on a fix based on the 5.7.2 tag
The master branch has some bugs to fix, so were not releasing any official releases based on that at the moment

@@ -217,10 +217,12 @@ async def get_user_str_from_post(self, scrape_item: ScrapeItem) -> str:
return soup.select_one("a[class=post__user-name]").text

async def get_user_str_from_profile(self, scrape_item: ScrapeItem) -> str:
"""Gets the user string from a scrape item."""
"""Gets the user string from a scrape item"""
service, user = await self.get_service_and_user(scrape_item)
Copy link

@lerroynick lerroynick Nov 27, 2024

Choose a reason for hiding this comment

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

remove await. await cant return tuples, and get_service_and_user is has not anything asyncable anyway :)
Then it works.

@jbsparrow
Copy link
Owner

#326 Fixes this & Kemono. Took a bit long I got distracted 😅

@jbsparrow jbsparrow closed this Nov 27, 2024
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.

[BUG] AttributeError: 'NoneType' object has no attribute 'text'
4 participants