-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
feat: use vite testing #315
base: main
Are you sure you want to change the base?
Conversation
@a2937 are you still working on this PR? i wanted to ask something if you do... |
What's the question? Edit: I had gotten stuck on this PR due to Netlify issues. I am not abandoning this PR. I just think I need a little help. |
How about also move from using create-react-app and move to use vite
instead?
…On Tue, Oct 29, 2024, 14:36 Anna ***@***.***> wrote:
@shootermv <https://github.com/shootermv>
What's the question?
Edit: I had gotten stuck on this PR due to Netlify issues. I am not
abandoning this PR. I just think I need a little help.
—
Reply to this email directly, view it on GitHub
<#315 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAKGMHWQ56I3C32QNQKQZ4LZ556MXAVCNFSM6AAAAABOEAB2GCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBUGA4DMOBYGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
What the netlify issues you facing?
…On Tue, Oct 29, 2024, 14:45 Moshe Vilner ***@***.***> wrote:
How about also move from using create-react-app and move to use vite
instead?
On Tue, Oct 29, 2024, 14:36 Anna ***@***.***> wrote:
> @shootermv <https://github.com/shootermv>
>
> What's the question?
>
> Edit: I had gotten stuck on this PR due to Netlify issues. I am not
> abandoning this PR. I just think I need a little help.
>
> —
> Reply to this email directly, view it on GitHub
> <#315 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAKGMHWQ56I3C32QNQKQZ4LZ556MXAVCNFSM6AAAAABOEAB2GCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBUGA4DMOBYGI>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
|
57a7c42
to
bacd2a1
Compare
Except I did. I also needed Typescript. The netlify issues were updating the header rules and dealing the pages changed thing. P.S Sorry about the next few pushes. The package.json is taking me by surprise. |
i can see something is wrong with cypress test for firefox - is running without finish |
Remind me to update this to use Playwright after this PR gets merged. |
i think i have idea about solve "endless cypress run" issue... |
I have a crazy idea but I think I know why it's continuing forever. It's trying to load in the ChatBot that is linked to the Discord server. |
3bc4c1c
to
8e671ff
Compare
i changed the |
6d81c16
to
6b9b277
Compare
Checklist:
Update index.md
)Closes #301
It's a pretty huge migration but it adds both Typescript to the main files and Vite.