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

Past expanded answers do not collapse in the question player #511

Closed
BenHenning opened this issue Dec 4, 2019 · 0 comments · Fixed by #1456
Closed

Past expanded answers do not collapse in the question player #511

BenHenning opened this issue Dec 4, 2019 · 0 comments · Fixed by #1456
Assignees
Labels
Priority: Essential This work item must be completed for its milestone. Z-ibt Temporary label for Ben to keep track of issues he's triaged.

Comments

@BenHenning
Copy link
Member

No description provided.

@BenHenning BenHenning added this to the Alpha milestone Jun 23, 2020
@BenHenning BenHenning added Type: Bug Priority: Essential This work item must be completed for its milestone. labels Jun 23, 2020
rt4914 pushed a commit that referenced this issue Aug 24, 2020
…er (#1456)

* fix previous response collapse

* added correct import

* added test cases

* added test for question player

* fix ktlint

* update skill_id to test_skill_id

* added QuestionPlayerActivityLocalTest

* nit changes as suggested

* removed unused import

* failed test to check over CI

* revert failed test to check over CI

* updating dagger modules

* adding profile init in test

* fix merge issue

* added questionplayer test to CI

Co-authored-by: Ben Henning <[email protected]>
@BenHenning BenHenning added the Z-ibt Temporary label for Ben to keep track of issues he's triaged. label Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Essential This work item must be completed for its milestone. Z-ibt Temporary label for Ben to keep track of issues he's triaged.
Development

Successfully merging a pull request may close this issue.

3 participants