Skip to content
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

Fix events repeatableAdded/Removed not working when used in DOM template #432

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

j0ber
Copy link

@j0ber j0ber commented May 14, 2021

Amazing project!
We are using it with DOM templates and are unable to use the "repeatableAdded" and "repeatableRemoved" events.

@justin-schroeder
Copy link
Member

@j0ber somehow this PR slipped under my radar. Seems like a reasonable change — for some small faction of people though this will represent a breaking change. It seems we may need to emit both events camel case and kebab. Mind making those edits and re-submitting?

@j0ber
Copy link
Author

j0ber commented Jun 22, 2021

Sure no problem.

@j0ber
Copy link
Author

j0ber commented Jun 22, 2021

Still have the infamous "error: option '-k, --key ' argument missing" with cypress... Really don't know why...

david-mears-2 added a commit to usemast/vue-formulate that referenced this pull request Mar 2, 2022
See wearebraid#432 for details. At time
of writing the PR was still open and unapproved.
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.

2 participants