Skip to content

Migrate All Components to TS#23

Open
Yedidyar wants to merge 16 commits into
SgtTepper:feature/typescriptfrom
Yedidyar:main
Open

Migrate All Components to TS#23
Yedidyar wants to merge 16 commits into
SgtTepper:feature/typescriptfrom
Yedidyar:main

Conversation

@Yedidyar

@Yedidyar Yedidyar commented Jul 27, 2023

Copy link
Copy Markdown

the only file that not been migrated is bubble.js

@vercel

vercel Bot commented Jul 27, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
beta-knesset ❌ Failed (Inspect) Aug 19, 2023 5:04pm

@Yedidyar Yedidyar changed the title Continue migrating to ts Migrate All Components to TS Jul 27, 2023
@thebne

thebne commented Aug 8, 2023

Copy link
Copy Markdown
Collaborator

היי @Yedidyar , אני בעומס בעבודה עד שבוע הבא, מבטיח להגיע לזה בהקדם וגם לעשות כבר merge למאסטר 😁

@Yedidyar

Yedidyar commented Aug 8, 2023

Copy link
Copy Markdown
Author

מעולה 💫

@thebne thebne left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

went through half of it, added some comments for now.

were you able to test it with actual data? there were some minor things I noticed that could potentially break the intended logic.

Comment thread src/pages/Quotes.tsx
Comment thread src/pages/Main/Search/Histogram.tsx Outdated
Comment thread src/pages/Main/Search/Histogram.tsx Outdated
Comment thread src/pages/Main/Search/Histogram.tsx
Comment thread src/pages/Main/PersonProfile/index.tsx
Comment thread src/pages/Main/PersonProfile/index.tsx Outdated
Comment thread src/pages/Main/PersonProfile/index.tsx Outdated
@Yedidyar

Copy link
Copy Markdown
Author

I tested it from time to time i will check again when i will resolve the comments.

@Yedidyar

Copy link
Copy Markdown
Author

the ci do not pass but i do not have access to the logs

@SgtTepper

SgtTepper commented Aug 19, 2023

Copy link
Copy Markdown
Owner

Error: Node.js Version "14.x" is discontinued and must be upgraded. Please set Node.js Version to 18.x in your Project Settings to use Node.js 18. @Yedidyar

@Yedidyar

Copy link
Copy Markdown
Author

It seems that the settings in the cicd project need to be changed

@Yedidyar

Copy link
Copy Markdown
Author

It seems that the settings in the cicd project need to be changed

He i think i cant fixed at my end
Can you check it out in the deployment config
@SgtTepper
@thebne

P.S can you review the rest of the code i would like to finish this migration and start working on more issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants