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

Add new question feature #1

Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

Add new question feature #1

wants to merge 30 commits into from

Conversation

weiquu
Copy link
Owner

@weiquu weiquu commented Jun 18, 2022

What is the purpose of this pull request?

  • Documentation update
  • Bug fix
  • Feature addition or enhancement
  • Code maintenance
  • DevOps
  • Improve developer experience
  • Others, please explain:

Overview of changes:
Add a new question type, the multiple blanks question type. The syntax is very similar to other question types. An example is provided below:

<question type="multiBlanks" header="This question _______ is a '_______ blanks' question.">
  <q-option keyword="type"></q-option>
  <q-option keyword="multiple" reason="That's the name of the question type."></q-option>
</question>

The result will look like this:
MultipleBlanksQuestionType

Anything you'd like to highlight/discuss:
Still some things to be added, such as support for multiple keywords, setting a threshold, and making blanks inline with the question.

Testing instructions:

Proposed commit message: (wrap lines at 72 characters)
Add new question type for questions with multiple blanks


Checklist: ☑️

  • Updated the documentation for feature additions and enhancements
  • Added tests for bug fixes or features
  • Linked all related issues
  • No unrelated changes

Neo Wei Qing and others added 30 commits June 18, 2022 10:42
Add a new question type, the "multiple blanks" question type. Still
some things to be added, such as support for multiple keywords,
setting a threshold, and making blanks inline with the question.
Merging to remove package-lock.json changes
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.

1 participant