-
Notifications
You must be signed in to change notification settings - Fork 42
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
Implement mentor availability endpoint #64
Implement mentor availability endpoint #64
Conversation
…into get-all-users-endpoint
…into get-all-users-endpoint
…21/scholarx-backend into get-all-users-endpoint
I havent written the tests yet. Wanted to make sure the endpoint is fine. Will do once approved. Also im not sure about the route @anjula-sack |
src/routes/profile/profile.route.ts
Outdated
profileRouter.put( | ||
'/mentor/availability', | ||
requireAuth, | ||
mentorAvailabilityHandler | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should go to mentor routes @Shrenik0321
src/routes/admin/admin.route.test.ts
Outdated
const defaultUser = { | ||
email: `test${randomString}@gmail.com`, | ||
password: '123' | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can import this from mock.ts.
src/routes/admin/admin.route.test.ts
Outdated
const randomString = Math.random().toString(36) | ||
const randomStringAdmin = Math.random().toString(36) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be removed. Check mock.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! @Shrenik0321
Purpose
The purpose of this endpoint is for the mentor to update their availability
Goals
Approach
Screenshots
Checklist
Related PRs
Test environment
Learning