search

kaiodeodato / pic_to_text

pic_to_text
https://kaiodeodato.github.io/pic_to_text/
0 stars 0 forks source link

Migrate from CRA to Vite #2

Open Zenb0t opened 1 year ago

Zenb0t commented 1 year ago

Hey Kaio. Create React App has a lot of vulnerabilities, and it's painfully slow. We should think about migrating it to something like Vite.

Here is a migrating tut, and more about Vite here

I can help you with this but should be fairly straightforward.

GIT-Gizmo commented 12 months ago

Hi, I'm happy to help with this, you can assign the issue to me if you don't mind

kaiodeodato commented 12 months ago

Hi man, cool! will be a honor! send me your linkedin for us to be in touch!

GIT-Gizmo commented 12 months ago

My pleasure.

Here is my linkedIn profile: https://linkedin.com/in/boluwatife-david

GIT-Gizmo commented 12 months ago

I have migrated to vite but the console return an error related to your supabase_url and supabase_key which isn't there you might have to put those in yourself. I also noticed you added your .env to gitignore which might also play a role in the error.

Zenb0t commented 12 months ago

Hey, sorry for jumping in, but this might be related to a few key differences between Vite and CRA. Importing env variables is different when using Vite. Here is the docs regarding the difference. Have to insert VITE_ in front of the env vars that you want to expose. And import then as import.meta.env.variable_name

GIT-Gizmo commented 12 months ago

Yeah, I'm happy to get feedback and tips on how to solve the issue. I'll check the doc for solution, thanks 👍

kaiodeodato commented 12 months ago

Hello David, I think the problem is because the supabase database is gone, I had to delete because I was with the maximum of two project free already. I need to come up with a solution that doesn't use Supabase to save the photo. do you have any idea?

kaiodeodato commented 12 months ago

I cleaned it up to make things clearer, main is in CRA and the backend is already corrected

GIT-Gizmo commented 12 months ago

Oh alright, sorry for not getting back to you soon enough, I was really busy today.

kaiodeodato commented 12 months ago

No problem, you will see that the repo restarted, but the code, now, is corrected 💪

Em sex., 1 de set. de 2023 às 21:39, David Bolu @.***> escreveu:

Oh alright, sorry for not getting back to you soon enough, I was really busy today.

— Reply to this email directly, view it on GitHub https://github.com/kaiodeodato/pic_to_text/issues/2#issuecomment-1703296368, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3REYNYXOYYO7RVQXYDMHHDXYJBYTANCNFSM6AAAAAAVS2INEU . You are receiving this because you commented.Message ID: @.***>