Skip to content
This repository has been archived by the owner on Dec 27, 2020. It is now read-only.

Mess Manager Functionality #40

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Mess Manager Functionality #40

wants to merge 4 commits into from

Conversation

lav-joshi
Copy link
Member

List of developments

  • Super User can add Mess Managers .
  • Development of Mess Manager Dashboard.

@lav-joshi lav-joshi requested a review from rishabhshuklax June 30, 2020 00:34
@lav-joshi
Copy link
Member Author

Sir today I will add some more functionality to mess manager's dashboard

@rishabhshuklax
Copy link
Member

rishabhshuklax commented Jun 30, 2020

So far this loks good!

Copy link
Member

@rishabhshuklax rishabhshuklax left a comment

Choose a reason for hiding this comment

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

while testing it locally I got this error

UnhandledPromiseRejectionWarning: ValidationError: User validation failed: year: Path `year` is required., rollNo: Path `rollNo` is required.
    at new ValidationError (/home/rs007/nodeJs/Hostel-Managment-Poral/node_modules/mongoose/lib/error/validation.js:31:11)

Could you also look into this.

@lav-joshi
Copy link
Member Author

Thank U sir . I will look into this error and make mess manager's dashboard more functional like to see how many students are present are not ,etc ?

@rishabhshuklax
Copy link
Member

What the status @lav-joshi ? Is it ready to merge or there's some pending work with it?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants