pFad - Phone/Frame/Anonymizer/Declutterfier! Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

URL: http://github.com/doubtfire-lms/doubtfire-api/pull/462

/> feat/add-requirement-api by Duy-Nguyen1104 · Pull Request #462 · doubtfire-lms/doubtfire-api · GitHub
Skip to content

feat/add-requirement-api#462

Open
Duy-Nguyen1104 wants to merge 4 commits into
doubtfire-lms:new/course-flowfrom
Duy-Nguyen1104:feature/courseflow
Open

feat/add-requirement-api#462
Duy-Nguyen1104 wants to merge 4 commits into
doubtfire-lms:new/course-flowfrom
Duy-Nguyen1104:feature/courseflow

Conversation

@Duy-Nguyen1104
Copy link
Copy Markdown

Description

This PR is built on top of feat/template-coursemap-api-updated. Please review that PR first.

This pull request introduces the Requirement feature to the CourseFlow API, as I noticed that a requirement entity is not created based on this doc requirement: https://thoth-tech.netlify.app/products/courseflow/documentation/design/backend-functional-requirements/#requirements.
It adds a new Requirement model, migration, and API routes for creating, retrieving, updating, and deleting requirements.
Relevant tests have also been created or updated to ensure the correctness of the new endpoints and model logic.

Type of change

  • [x ] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Added and ran unit tests for the new Requirement model and API endpoints.
  • Checked that all existing and new tests pass locally
  • Verified API routes using automated request specs and manual testing via Postman
  • Confirmed database migration applies cleanly and schema is updated.

image

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation if appropriate
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

pFad - Phonifier reborn

Pfad - The Proxy pFad © 2024 Your Company Name. All rights reserved.





Check this box to remove all script contents from the fetched content.



Check this box to remove all images from the fetched content.


Check this box to remove all CSS styles from the fetched content.


Check this box to keep images inefficiently compressed and original size.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy