Conversation
|
@GamerC0der is attempting to deploy a commit to the Karo Roghzai's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Hi. I reviewed the PR. You can actually check a preview here, though I'm not sure if you have permissions to see it. First off, this is a very big PR. Too big to just be a patch. If this were to be merged, it'd be a minor update, not patch (it would be 0.10.0, not 0.9.2). Second, this fails a lot of checks:
Example 2 (settings):
6. You changed the image rate limit from 12 to 8 when it should remain 12. I don't get why?
At the moment, I think those are all of the issues you may need to address IF you want to keep this PR as large as it is. I'll happily merge smaller PRs like, say, only adding a couple new models or adding that usage indicator UI (which I really love!) I really like the ideas in this PR though. It's clear you're vibecoding it, just please make sure to test every main feature of your PR (or clearly address any issues in the PR itself) thoroughly before commiting/making a PR so that we don't run into issues such as the Max mode barely being functional or not working on mobile screens. |


Max Mode & Deepseek R1 0528