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

enhancement-implemented #56

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

agrim0312
Copy link

changed - Link to CustomLink in ./common.tsx due to conflict of same names

Screencast.from.17-12-23.05.04.10.PM.IST.webm

however, did not delete the share page file

Copy link

vercel bot commented Dec 17, 2023

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

Name Status Preview Comments Updated (UTC)
amica ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 17, 2023 0:53am

@agrim0312
Copy link
Author

fixed #52

@kasumi-1
Copy link
Contributor

Hi @agrim0312 thank you. However, can't merge yet as it looks like this broke the build. You can check locally by running npm run build to run the typechecker.

@agrim0312
Copy link
Author

sure

@agrim0312
Copy link
Author

@kasumi-1 fixed the build You can review it now

@kasumi-1
Copy link
Contributor

This freezes the character when closed. Also, there are some whitespace and indent issues - I fixed these in https://github.com/semperai/amica/tree/sharePage-2-modal-component but I am unsure of cause of freezing on close.

@agrim0312
Copy link
Author

agrim0312 commented Dec 18, 2023

hi @kasumi-1, I am not able to solve the issue however I have narrowed it down. As soon as VrmDemo gets loaded in the shareModal.tsx file the character freezes. I tried rendering modal without VrmDemo and it did'nt freeze and worked fine.
Screenshot from 2023-12-18 17-18-11
here is the VrmDemo div. Please guide me so that I can fix the issue.

@slowsynapse
Copy link
Collaborator

I think this PR is resolved with the load/save VRM feature added. Will double check in future.

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.

None yet

3 participants