-
Notifications
You must be signed in to change notification settings - Fork 19
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
Template sweep -- fix bugs in Vue and Lit #130
Conversation
Prevent trailing commas when field list ends in non-widget fields in EditFoo.svelte template
@pdaoust Same with this one, please can it target I'm happy reviewing the code but I don't know the repo well enough to understand what's changing without having a conversation about it if that's okay! :) |
56cb4dd
to
3ca29ed
Compare
This is now ready for review and can be merged when ready |
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.
Discovered similar bugs, at least for the Vue
and Svelte
templates hence why I am returning to this PR, I think we should be getting this PR merged as it does address some critical issues
@pdaoust kindly let me know if you are in a position to resolve the merge conflicts
…in/scaffolding into 2023-10-03-fix-vue-lit-templates
@pdaoust, allow me to close this PR since it is stale at the moment and most of the changes made here have long been addressed |
Sure @c12i . Are there any fixes that haven't already been made? Should I do a new PR with those? |
Thanks @pdaoust, I don't think there are any changes at the moment, I inadvertently addressed them while transitioning the templates to the new styling. |
I don't claim this is a thorough sweep that catches all bugs, but it does catch all bugs discovered via following the Getting Started Guide but choosing Lit and Vue instead of Svelte for the templating engine.