-
-
Notifications
You must be signed in to change notification settings - Fork 121
Conversation
added myself as a contributor underneath the credits section
@YourMobileGeek Could we optimize the image for performance before it gets merged? Making it a smaller width and making it ~500KB in size would be best. We are adding an integration that will do this automatically but for now, that would be ideal! |
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.
I would optimize the image to be smaller since it's 4.2MB and that's it - thank you for your PR! 😄
@fvcproductions Sounds good, I can go ahead and look into making it a smaller width, it regards to optimization to 500kb is there a workaround around for this you would suggest? |
@fvntr oh wow, yea I definitely overlooked that haha. I was thinking that you wanted to use another integration for the time being. Thanks! I'll go ahead and switch out this image and do a pull request. |
…Geek/website into update_homepage_image
I ahead and switched out the homepage image provided through Google Photos for the Techqueria website.
✅️ By submitting this PR, I have verified the following