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

Solved bug, adding two fields containing typeUserAttr, the typeUserAt… #258

Merged
merged 1 commit into from
Sep 21, 2016

Conversation

gimigliano
Copy link
Contributor

@gimigliano gimigliano commented Sep 21, 2016

Solved bug, adding two fields containing typeUserAttr, the typeUserAttr of the second field had the values of the typeUserAttr of the previous field

…tr of the second field had the values of the typeUserAttr of the previous field
@kevinchappell
Copy link
Owner

This looks good to me but can you describe the steps to reproduce the error? I tried a few combinations but was not able to produce the error.

@gimigliano
Copy link
Contributor Author

In this fiddler https://jsfiddle.net/ab5a3sd0/1/ try to add a new text field.
You'll find the "myfield" attribute valorized with the value of the formData text field

@kevinchappell
Copy link
Owner

@gimigliano thanks, thats clears it up.

@kevinchappell kevinchappell merged commit b0203b3 into kevinchappell:master Sep 21, 2016
@kevinchappell
Copy link
Owner

kevinchappell commented Sep 21, 2016

@gimigliano btw, youre killing it on these PRs, you even made it on the top contributors list, good work 🎉

@gimigliano
Copy link
Contributor Author

Wow, what have I win?

@kevinchappell
Copy link
Owner

@gimigliano 💯 internet points 😄

@gimigliano
Copy link
Contributor Author

gimigliano commented Sep 21, 2016

oh I'm so happy. Just remember to add typeUserAttr in README.md options

@kevinchappell
Copy link
Owner

knew i was forgetting something... Should be there now.

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