Tushar Reddy
Tushar Reddy
@Kitenite Would you like me to set up a PR for this? The change itself is trivial, but this reduces a lot of console errors being logged during local development,...
Hi, I spent some time today looking at the image implementation both on the server/client side. I was wondering what kind of an improvement you were looking for, pretty simple...
@Kitenite Sounds good! I'll start with just implementing the image limit (set to 5) in one PR, then we can discuss the compression enhancement as a follow-up. For the configuration,...
@Kitenite I spent a few hours today doing some stress testing with the image upload failures. Uploaded batches of 15-16MB screenshots and documented the error progression through console logs. Hit...
> Merged the first PR! [#2706](https://github.com/onlook-dev/onlook/pull/2706) [@reddygtvs](https://github.com/reddygtvs) awesome work! Thank you for helping me along the way :) - more to come!
Will take up this issue. I actually ended up facing some other bug related to screenshotting which I'll try to fix on the same commit. (2 identical screenshots being taken...
@Kitenite Hi Kiet! It's been a while since I touched this branch. Should I continue working on this by fixing the conflicts due to the more recent changes or leave...
For the ECMAScript part, is this similar to the error you faced? I believe this and #2734 are in the same category. I'll work on the fix.