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

added endpoints,apidoc and testcases #360

Merged
merged 5 commits into from
Oct 14, 2023

Conversation

hricha11
Copy link
Contributor

Description

Briefly describe the changes made in this pull request.

Fixes

Fixes #354

Checklist

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

Copy link
Member

@TejasNair9977 TejasNair9977 left a comment

Choose a reason for hiding this comment

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

add a registry in app.js for this, so that testcases pass

@demonkillerr
Copy link
Member

please make sure you are pulling (making your local up-to-date) before pushing

Copy link
Member

@TejasNair9977 TejasNair9977 left a comment

Choose a reason for hiding this comment

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

LGTM

@TejasNair9977 TejasNair9977 merged commit cd837dc into development Oct 14, 2023
1 check passed
@TejasNair9977 TejasNair9977 deleted the 354-attendance-endpoints-apidoc-testcases branch October 15, 2023 16:57
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.

3 participants