Skip to content
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

resolving errors #19

Merged
merged 1 commit into from
Apr 3, 2024
Merged

resolving errors #19

merged 1 commit into from
Apr 3, 2024

Conversation

Aditya-eddy
Copy link
Member

Fixes the Building errors because of LottiePlayer:

image

Signed-off-by: Aditya-eddy <[email protected]>
@Aditya-eddy
Copy link
Member Author

Hey @nehagup i'll attach the screenshot:
Screenshot from 2024-04-03 10-59-04

@nehagup nehagup requested a review from Hermione2408 April 3, 2024 05:57
@Hermione2408
Copy link
Member

Hermione2408 commented Apr 3, 2024

Fixes the Building errors because of LottiePlayer:

image

@Aditya-eddy In which node version are you getting these errors, I tried building the main branch in node 18 didn't get any error

@Aditya-eddy
Copy link
Member Author

Aditya-eddy commented Apr 3, 2024

Basically this error is caused by node>18. I am using node v-20.
Its because of the Lottie player.
Ref

@Hermione2408
Copy link
Member

Basically this error is caused by node>18. I am using node v-20. Its because of the Lottie player. Ref

Got it!

Copy link
Member

@Hermione2408 Hermione2408 left a comment

Choose a reason for hiding this comment

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

LGTM! @nehagup

@nehagup nehagup merged commit 96b0f55 into keploy:main Apr 3, 2024
1 check passed
@Aditya-eddy Aditya-eddy deleted the reducingBuildErrors branch April 4, 2024 23:44
vibhaw1904 pushed a commit to vibhaw1904/keploy-website that referenced this pull request Apr 17, 2024
Signed-off-by: Aditya-eddy <[email protected]>
Signed-off-by: vibhaw raj <[email protected]>
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