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 removal of error messages #790

Merged
merged 1 commit into from May 2, 2020
Merged

Conversation

tagliala
Copy link
Contributor

@tagliala tagliala commented May 2, 2020

The previous implementation was searching for input_tag classes on the
label field wrapper, resulting in the element itself not being removed
And the old label not being added.

This commit fixes the wrong behavior

The previous implementation was searching for `input_tag` classes on the
label field wrapper, resulting in the element itself not being removed
And the old label not being added.

This commit fixes the wrong behavior
@coveralls
Copy link

coveralls commented May 2, 2020

Coverage Status

Coverage remained the same at 100.0% when pulling fb11038 on bugfix/fix-label-error-classes into a49dd74 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 99.802% when pulling fb11038 on bugfix/fix-label-error-classes into a49dd74 on master.

@tagliala tagliala merged commit 23a218c into master May 2, 2020
@tagliala tagliala deleted the bugfix/fix-label-error-classes branch May 9, 2021 09:49
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.

None yet

2 participants