oppia / oppia-android

A free, online & offline learning platform to make quality education accessible for all.
https://www.oppia.org
Apache License 2.0
309 stars 509 forks source link

Answers in the options are displayed incorrectly but on submitting answers they are displayed correctly #4331

Open KolliAnitha opened 2 years ago

KolliAnitha commented 2 years ago

Describe the bug Answers in the options are displayed incorrectly but on selecting they are displayed correctly in The commutative and associative laws

To Reproduce Steps to reproduce the behavior:

  1. Go to 'All Topics' - Expressions and Equations
  2. Complete 1st Chapter
  3. In the 2nd chapter- The Commutative and Associative Laws' go to question 532+7
  4. Verify the options displayed

Expected behavior 37 must be displayed instead of 3 for 1st and 3rd options

Actual Result: 3 is displayed as a product for 1st and 3rd options

Screenshot_2022-04-28-11-26-26-05_943a62cb4c6fb83e010e1c2e82766a17.jpg

Environment Device name: One plus Nord2 5G Android version : Android 11 App version : 0.7-alpha-d1b2827517

After submitting the answers the products are displayed correctly

Screenshot_2022-04-28-11-34-15-59_943a62cb4c6fb83e010e1c2e82766a17.jpg

BenHenning commented 2 years ago

Ah, I'm curious how this'll change with #3852 merged in as it's making a lot of changes to the bullet list.

Either way, this seems like a low-priority fix as the content isn't actually wrong. I think we'd be fine shipping this in beta.

BenHenning commented 2 years ago

I think we can wait until #3852 is merged before investigating this.

BenHenning commented 2 years ago

For searching context, this issue was found in 0.7-alpha (MR4).

BenHenning commented 1 year ago

Now that #3852 has been merged, can you check this again @KolliAnitha to see if it's still an issue?

KolliAnitha commented 1 year ago

@BenHenning, Topic: Expressions and Equations is not available in the beta version as per https://github.com/oppia/oppia-android/issues/4435

BenHenning commented 1 year ago

Ah good catch. We'll need to check this with the next alpha push.