Skip to content

[ADDED]added crud, all endpoints, apidocs , testcases for activity. #341

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
Merged
merged 4 commits into from
Sep 30, 2023
Merged

[ADDED]added crud, all endpoints, apidocs , testcases for activity. #341

merged 4 commits into from
Sep 30, 2023

Conversation

@deepti-50
Copy link
Contributor

@deepti-50 deepti-50 commented Sep 12, 2023 •
edited
Loading

Description

  1. Added crud operation for actvity.js in models folder.
  2. Created endpoints for all operations in /routes/activity.js , /controller/activity.js , /services/activity.js
  3. Created endpoints for /activity/add , /activity/update , /activity/delete/:id , /activity/list.
  4. Made an entry for activity in _apidoc.js folder.
  5. Created testcases for every route that has been created.

Fixes

Fixes the issue #315 .

Checklist

  • Code follows project's style guidelines.
  • Changes are documented appropriately.

(PS → Make Sure Pull request title is meaningful.)

kskarthik05
kskarthik05 approved these changes Sep 30, 2023
View reviewed changes
Copy link
Contributor

@kskarthik05 kskarthik05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Aastha-S-Rai Aastha-S-Rai merged commit c76251a into development Sep 30, 2023
@TejasNair9977 TejasNair9977 deleted the 315-crud-endpoint-activity branch October 15, 2023 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kskarthik05 kskarthik05 kskarthik05 approved these changes

@TejasNair9977 TejasNair9977 Awaiting requested review from TejasNair9977

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@deepti-50 @kskarthik05 @Aastha-S-Rai @TejasNair9977