Skip to content

Conversation

RubberRoman
Copy link
Contributor

Scraper type(s)

  • performerByName
  • performerByFragment
  • performerByURL
  • sceneByName
  • sceneByQueryFragment
  • sceneByFragment
  • sceneByURL
  • groupByURL
  • galleryByFragment
  • galleryByURL
  • imageByFragment
  • imageByURL

Examples to test

Short description

Support all sites from MyMemberSite network.

@RubberRoman RubberRoman force-pushed the feature/MyMemberSite branch from a826154 to 3c3fbf4 Compare October 4, 2025 21:59
@RubberRoman RubberRoman force-pushed the feature/MyMemberSite branch from 3c3fbf4 to 9bb5936 Compare October 4, 2025 22:04
Copy link
Contributor

@smith113-p smith113-p left a comment

Choose a reason for hiding this comment

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

I stopped looking because the list is so long..

@feederbox826
Copy link
Collaborator

feederbox826 commented Oct 5, 2025

Ran it through n++, there is ONE duplicate line still 😭

Edit, it was python3 thank god

@feederbox826
Copy link
Collaborator

with so many lines, this might actually cause problems for the parsers, I don't think it makes sense to list out all the domain paths underneath mymember.site, it would be easier just to do mymember.site and allow both /videos/ and /photosets/ underneath both scrapers since they're URL based, I don't think theres much value to having them all mapped out

@RubberRoman
Copy link
Contributor Author

@feederbox826 Is it better now? Now I used only domains and referenced the URL list to reduce the number of lines in the scraper definition.

@RubberRoman RubberRoman force-pushed the feature/MyMemberSite branch from b79b800 to b233cd4 Compare October 5, 2025 08:38
@DogmaDragon
Copy link
Contributor

It looks fine at a glance now, but I haven't tested it.

@feederbox826
Copy link
Collaborator

Busy with life and got lost in my head, looks good at a glance, will try to give it a test now

Copy link
Collaborator

@feederbox826 feederbox826 left a comment

Choose a reason for hiding this comment

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

validate scene aginst base and custom domain

@feederbox826 feederbox826 merged commit 1e70086 into stashapp:master Oct 13, 2025
1 check passed
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.

4 participants