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

Exam answer issue #2651

Closed
lmmrssa opened this issue Dec 5, 2018 · 7 comments
Closed

Exam answer issue #2651

lmmrssa opened this issue Dec 5, 2018 · 7 comments
Assignees

Comments

@lmmrssa
Copy link
Member

lmmrssa commented Dec 5, 2018

After creating exam with single choice, long text then multi choice
It disabled submit at last question (multi choice) then moving back to old question does not show previous answer
screen shot 2018-12-05 at 07 57 44
screen shot 2018-12-05 at 07 58 06

@paulbert
Copy link
Member

paulbert commented Dec 5, 2018

@lmmrssa This is two separate issues, I think. Initializing the question with the previously filled in answer hasn't ever worked, but we should make sure exams of different configurations are able to be finished first.

@paulbert
Copy link
Member

paulbert commented Dec 5, 2018

Error is below. My guess is that when switching questions this.answer isn't reset from the markdown. Tested with the other three question types before the multiple answer and they all worked.

image

@nadaa nadaa self-assigned this Dec 6, 2018
@nadaa
Copy link
Member

nadaa commented Dec 6, 2018

Also I got this error when submitting:

@nadaa
Copy link
Member

nadaa commented Dec 9, 2018

@paulbert, I have checked this, it's a binding issue in markdown, it's only one-way. Any suggestion how to solve this?

@paulbert
Copy link
Member

@nadaa I think this will actually be fixed in #2667. It's not a binding error. We use two way binding on the markdown component in a more verbose format. See https://angular.io/guide/template-syntax#inside-ngmodel for an example, but I'll try to explain here, too:

The simple

[(...)]="variable"

can be written as

[...]="variable" (...Changes)="function"

when you need to do more than just update the value of the variable when it changes.

@paulbert
Copy link
Member

Update above: #2667 only fixes showing the previous answer. Does not fix long text -> multiple answer breaking issue.

paulbert added a commit that referenced this issue Dec 11, 2018
@paulbert
Copy link
Member

2nd update: #2667 now fixes this as well.

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

No branches or pull requests

3 participants