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

Bug: Add Form Clears Title and Leaves Labels #919

Closed
GeekSage opened this issue May 20, 2014 · 4 comments · Fixed by #923
Closed

Bug: Add Form Clears Title and Leaves Labels #919

GeekSage opened this issue May 20, 2014 · 4 comments · Fixed by #923
Labels

Comments

@GeekSage
Copy link

(This is for the 1.9.0 release)

Sorry, the title isn't that helpful but here's how to reproduce the bug I noticed. Go to Manage - Forms and create a new form (I entered the following):
Title = Test
Label = Test - Short Answer - Variable (testa)
Label = Test2 - Short Answer - Variable (testb)

After entering the above values I clicked the "Add Form" button. At this point the title is wiped out but the label values still remain. If I then enter another title (different than the first) and click "Add Form" again it will then wipe out the label values as well as the title but on the newly created form the two label values will not exist.

I'm guessing the desired behavior should be one of the following:

  1. Clear all values leaving the user able to enter a completely new form
  2. Leave the values but enter an "edit" mode for the newly created form
@greezybacon greezybacon added bug and removed forms labels May 20, 2014
@greezybacon
Copy link
Contributor

Good catch. Thanks

@greezybacon
Copy link
Contributor

Can you tell if the patch in #919 fixes the issue?

@GeekSage
Copy link
Author

That worked perfectly! What is the standard procedure with this type of thing? Am I supposed to close this issue out at this point?

@greezybacon
Copy link
Contributor

I generally prefer for you (the submitter) to confirm the fix, then GitHub will auto-close this issue when we merge the accompanying pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants