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.

Sign up for GitHub

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

Jump to bottom

Quiz sequencing questions name does not strip html tags. #439

Closed
garemoko opened this issue Feb 15, 2019 · 2 comments
Closed

Quiz sequencing questions name does not strip html tags. #439

garemoko opened this issue Feb 15, 2019 · 2 comments
Labels
fix released

Comments

@garemoko
Copy link
Contributor

Description
Quiz sequencing questions name does not strip html tags.

Version
v3.18.3

Steps to reproduce the bug
Answer a quiz containing a sequencing question (other question types may also be affected)

Expected behaviour

  • html tags should be stripped from quiz question names because the object,definition.name property is supposed to contain plain text.

Actual behaviour

  • html tags are not removed.
@ryasmi
Copy link
Member

ryasmi commented Feb 18, 2019 •
edited
Loading

Agreed. I think there is a util for stripping tags.

@ryasmi ryasmi added the fix label Feb 18, 2019
@garemoko garemoko mentioned this issue Feb 21, 2019
Merged
ryasmi pushed a commit that referenced this issue Feb 21, 2019
@garemoko @ryasmi
fix: Strips HTML from quiz names to fix #439. (#446 - Thanks @garemoko)
5853986
@HT2Bot
Copy link
Member

HT2Bot commented Feb 21, 2019

🎉 This issue has been resolved in version 4.2.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fix released
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@garemoko @ryasmi @HT2Bot