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

Fixed accessibility labels are not always created in problems. #3451

Merged

Conversation

waheedahmed
Copy link
Contributor

In multiple part question the label did not show up as a "label" field for every second part. e.g.(only showing for 1st, 3rd, 5th...)

JIRA: STUD-1524

@waheedahmed
Copy link
Contributor Author

@andy-armstrong
Copy link
Contributor

I'm nervous about this, so can you explain how your change is working. The code you removed said it was there to handle problems with multiple questions, and then it looks like you have also removed the test which verified that case. I wouldn't expect to see any tests removed as part of this change.

Also, please add a new test for the problem you have fixed.

@waheedahmed
Copy link
Contributor Author

@andy-armstrong the code which i removed actually creating this bug and the tests are written for the scenario which it was creating, so i fixed them too.

@davestgermain
Copy link
Contributor

@waheedahmed it looks like the test you removed covers the case where somebody has multiple questions but decided only to put the label on the first question. I believe that removing the code in edit.coffee will have the effect of assigning the label to the wrong question, in this case. Is that true?

@waheedahmed
Copy link
Contributor Author

@davestgermain @andy-armstrong does it looks fine now?

@davestgermain
Copy link
Contributor

that edit.coffee function is probably in need of a rethink, but for now 👍

@waheedahmed
Copy link
Contributor Author

@andy-armstrong waiting for thumbs up.

@andy-armstrong
Copy link
Contributor

👍 Sorry to be slow getting back to you, and thanks for addressing my concerns.

waheedahmed added a commit that referenced this pull request Apr 30, 2014
…s-missing-in-problems

Fixed accessibility labels are not always created in problems.
@waheedahmed waheedahmed merged commit 37d3d4a into master Apr 30, 2014
@waheedahmed waheedahmed deleted the waheed/stud1524-accessibility-labels-missing-in-problems branch April 30, 2014 08:56
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.

4 participants