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

Image Scraping Proof of Concept #3

Closed
wants to merge 1 commit into from
Closed

Conversation

colbyfayock
Copy link
Owner

No description provided.

@doingandlearning
Copy link
Collaborator

I really like this approach - here is another webpack module you're building - feels like there are a few of these in the project? Alongside the search idea ... love it!

@colbyfayock
Copy link
Owner Author

thanks. i started to get worried about it, i wasnt sure for intsance, if doing a ton of API requests to fetch the media objects would kill the server or at least start throttling 😂

i might look to see how gatsby's doing it

for now - i was consdering thinking of a way to add the wordpress CDN to the links 🤔 the photon one

https://developer.wordpress.com/docs/photon/

@colbyfayock colbyfayock mentioned this pull request Oct 1, 2020
@doingandlearning
Copy link
Collaborator

Interesting - an image CDN is great!

Does WP have a free CDN or would we push toward Cloudinary or similar?

@colbyfayock
Copy link
Owner Author

yeah it's that Photon link https://developer.wordpress.com/docs/photon/

the pattern is:

http://i0.wp.com/$REMOTE_IMAGE_URL

I'm not sure of the restrictions, but it would be technically coming from wordpress content

@colbyfayock
Copy link
Owner Author

i think hvaing documentation on how to use something like cloudinary would be valuable though - but im hoping to make the starter as service-free to start as possible, that way someone can get up and running with just a simple worpress site

@colbyfayock
Copy link
Owner Author

Closing this simply because it wouldn't be merged in "as is". it's linked in the issues to derive any value from it as a POC

@colbyfayock colbyfayock deleted the master+images branch February 25, 2021 18:40
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.

None yet

2 participants