Skip to content

Conversation

@joeizang
Copy link
Collaborator

Address #3238

@joeizang joeizang added the WIP label Dec 20, 2022
@joeizang joeizang self-assigned this Dec 20, 2022
@netlify
Copy link

netlify bot commented Dec 20, 2022

Deploy Preview for remixproject ready!

Name Link
🔨 Latest commit 3477c46
🔍 Latest deploy log https://app.netlify.com/sites/remixproject/deploys/63d92cc013dc030009c16a37
😎 Deploy Preview https://deploy-preview-3245--remixproject.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@joeizang joeizang marked this pull request as ready for review December 21, 2022 10:55
@joeizang joeizang requested a review from Aniket-Engg December 26, 2022 08:24
@joeizang joeizang linked an issue Dec 27, 2022 that may be closed by this pull request
@joeizang joeizang requested a review from yann300 December 28, 2022 14:43
@joeizang joeizang force-pushed the add-sol2Uml branch 2 times, most recently from c3de8ac to db1ec65 Compare January 2, 2023 09:22
@yann300
Copy link
Contributor

yann300 commented Jan 2, 2023

the flatten action doesn't do anything.

resolveContentAndSave: (url: string) => Promise<string>
fileExists: (file: string) => Promise<boolean>
writeFile: (file: string, content: string) => Promise<void>
writeFile: (file: string, content: any) => Promise<void>
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this change necessary?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, when left as a string, It becomes difficult to write the pdf to file explorer. Even though deep in the recesses of the plugin engin, it takes a string. Making it any allows me to write the result from generating the pdf to fileexplorer.

@joeizang joeizang requested a review from Aniket-Engg January 12, 2023 10:02
@joeizang joeizang force-pushed the add-sol2Uml branch 2 times, most recently from f0c45a1 to 68cc318 Compare January 19, 2023 08:44
@joeizang joeizang force-pushed the add-sol2Uml branch 2 times, most recently from 4790c76 to 6b23eb4 Compare January 20, 2023 15:40
@Aniket-Engg Aniket-Engg merged commit debbe62 into master Jan 31, 2023
@Aniket-Engg Aniket-Engg deleted the add-sol2Uml branch January 31, 2023 15:30
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.

move Sol2Uml to remix-project repo

5 participants