-
-
Notifications
You must be signed in to change notification settings - Fork 230
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
Convert loadMdx.server.js
to TypeScript
#691
Comments
I'd like to try this one! :) |
@sadiejay its all you! |
@sadiejay Are you still interested in this one? |
Sorry I dropped this! I'd be happy to continue to tackle it with some help!
But if you don't have the bandwidth to help me like before, then you can
pass it to someone else :)
…On Fri, May 26, 2023 at 12:17 PM Dan Ott ***@***.***> wrote:
@sadiejay <https://github.com/sadiejay> Are you still interested in this
one?
—
Reply to this email directly, view it on GitHub
<#691 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEVCCK7NE2L6XXLKFAAN4DDXIDJQTANCNFSM6AAAAAARBVGZDE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Expression of Interest: Convert
|
@Mohsin-mw go for it! |
Note for Hacktoberfest:
Virtual Coffee is reserving all open issues for Virtual Coffee members for the month of October.
Find out more about becoming a member
File
app/util/loadMdx.server.js
Issue Context
Used to load in mdx files in the remix app.
loadMdxDirectory
is a recursive function - it can call itselfBe sure to add some comments to the type definitions.
Check out this for an example of what I mean (example code compliments of @JoeKarow): https://github.com/Virtual-Coffee/virtualcoffee.io/blob/80028b4cba8b190d2931d8284f9278417e5ed82c/app/util/cmsimage.ts
Those comments can include markdown and we're generally following TSDoc - they result in nice things like this in code editors:
Blockers
The text was updated successfully, but these errors were encountered: