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
docs: add svelte and vue front-end SDKs #1760
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Coverage report
Show new covered files 🐣
Show files with reduced coverage 🔻
Test suite run success971 tests passing in 157 suites. Report generated by 🧪jest coverage report action from c6a142c |
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.
Super! Thanks for adding them 😄 I only have one small request as called out in the comment, but I trust you to fix that and then merge when you feel like it 🙌🏼
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.
Looks good aside from the folding admonition, but I think you might have fixed that locally already? I've marked it as a requested change regardless.
Were you also going to re-order the community SDKs by language?
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.
Nice one! One more change suggestion and it's good to go.
I also didn't realize we had that list of proxy sdks stored in that many places. We should definitely do something about that sometime 😅
Co-authored-by: Thomas Heartman <thomas@getunleash.ai>
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.
Great work! Let's get it in 🚂
Also re-ordered them and made them more consistent on the
bootstrapping
section.