Skip to content

Activity Questions CRUD Operations IV: Short Answer#186

Open
jasondsouza35 wants to merge 11 commits intomainfrom
josh-jason/crud-operations-short-answer
Open

Activity Questions CRUD Operations IV: Short Answer#186
jasondsouza35 wants to merge 11 commits intomainfrom
josh-jason/crud-operations-short-answer

Conversation

@jasondsouza35
Copy link
Copy Markdown
Collaborator

@jasondsouza35 jasondsouza35 commented Jan 29, 2026

Notion ticket link

Activity Questions CRUD Operations IV: Short Answer

Implementation description

  • Tested CRUD operations for the Input (Short Answer) question type

Steps to test

  1. For all 3 data types, (i.e short answer, numeric set, and numeric range) test CRUD. For Example:
POST

Endpoint - 
http://localhost:8080/activities/6775be2afa8027fddf428b44/TextInput

Body - 
{
  "questionText": "What is the capital of France?",
  "instruction": "Type your answer below",
  "placeholder": "e.g., Paris",
  "maxLength": 50,
  "validation": {
    "mode": "short_answer",
    "answers": ["Paris"]
  }
}

RETURNS - 
{
    "pages": [
        "697aad9230bcf50b4e02b52b"
    ]
}

What should reviewers focus on?

Note: You should replace the coursemodule/ coursepage id when testing in your own database. You should only be able to update an activity that is in the "Draft" or "Unpublished" states.

Reference the notion ticket for more testing details.

Checklist

  • My PR name is descriptive and in imperative tense
  • My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
  • I have run the appropriate linter(s)
  • I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

@jasondsouza35 jasondsouza35 marked this pull request as ready for review February 4, 2026 01:24
@ji-mmyliu ji-mmyliu self-requested a review February 25, 2026 00:32
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.

2 participants