-
Notifications
You must be signed in to change notification settings - Fork 148
fix all missing logos #2048
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 all missing logos #2048
Conversation
|
@benleoau I'd request that you not submit PRs to update "aggregator" scrapers unless it's to fix already included studios. This is why the "custom" scrapers option was created. Each person picks exactly which studios they want to scrape. Every time we change what's being scraped GLOBALLY from aggregate sites, we run the risk of complicating someone's existing setup (duplicating/overwriting/deleting/blah blah). If you want to share snippets of your "scrapers.json" for others to use as custom sites, I'd suggest you jump over and join us on discord. case in point, I'm missing previously scraped studios that were removed from SLR that I had downloaded & matched scenes for but they were removed from my library when someone removed the site from the scrapers.json |
|
good progression 4 updates to content |
|
100% |
pkg/config/scrapers.json
Outdated
| { | ||
| "url": "https://vrporn.com/studio/vrmassage/", | ||
| "name": "VR Massage", | ||
| "company": "VR Massage", |
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.
nice
|
I have added aggregator because there is no other options, none of them has the website, they are the only scraper that we can have. they are small studios that mostly does not have even website. |
|
HoloGirlsVR website has gone, we have to use aggregator, vrporn or slr. it mean even if you have HoloGirlsVR in your inctance, it is not working anymore. please have a look to their website https://www.hologirlsvr.com/ it is shut down |
|
benleoau:feature/addNewScrapers can you send me a windows version 4 me to test? |
|
vrplayful: website is a redirect to aggregators |
|
btw: I have found a bug in virtualrealporn.com scaper, it cannot find the image url and try to record whole json in the database which caused a crash in the app. I cannot put brackpoint in windows so I cannot debug it. this is my first time I am developing Go. #2049 |
|
I understand why you did, but you should just offer this as "custom scraper" snippets for users to add themselves. You are just "cluttering up" the scrapers list with studios from an aggregator that not everyone needs. |
I am totally confused, I can see lots of scarpers from VRPorn and SLR in codebase. I thought it is a normal approach to add more. is there any document how users can add custom scraper? I have edited my local scrapers.json but when I run the app, that file is going to over-write by the original one. |
|
no worries |
Don't be confused. Grin... The reason for all the existing studios was before everyone had the ability to do their own studios via the CUSTOM scrapers. Now it doesn't make since to update "official" studios on aggregators cuz you can add whichever ones YOU want without affecting anyone else's installation. |
|
I am going to say... Close this PR and open a new one with JUST the logo updates. Aggregators "studio" code will probably not get merged into release. Please read this PR on the configurable scrapers list. This negates the need for what you are trying to accomplish... Either add studios via the GUI (advanced/custom sites) or add them to the proper place in the scrapers.jason file. |
|
@theRealKLH |

No description provided.