-
Notifications
You must be signed in to change notification settings - Fork 2
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
Live demo #1
Live demo #1
Conversation
Cloned and squashed from https://github.com/makenew/tsmodule/releases/tag/v2.1.0 (makenew/tsmodule@88e4b26).
This reverts commit c54e804.
This reverts commit 80ea707.
This reverts commit c5ce4c6.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@razor-x good job! just a few small improvements/ comments
You must have Developer access to commit code to Seam API - Production on Vercel. If you contact an administrator and receive Developer access, commit again to see your changes. Learn more: https://vercel.com/docs/concepts/teams/roles-and-permissions#enterprise-team-account-roles |
Lets go! 🥳
I know this is huge, so if we want to break it up let's talk about it. But this is everything I've done so far.
And side note, since we haven't used the new package namespace yet I don't think we need to delete or reset any history unless there is something specific we are worried about.
I can annotate this on Monday if that helps before I mark it ready for review