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

Jatin create different question sets #1167

Open
wants to merge 3 commits into
base: development
Choose a base branch
from

Conversation

JatinAgrawal94
Copy link
Contributor

@JatinAgrawal94 JatinAgrawal94 commented Dec 7, 2024

Description

Please include the exact bug/functionality description and a summary of the changes/ related issues. Please also include any other relevant motivation and context:
339

Related PRS (if any):

To test this backend PR you need to checkout the #2928 frontend PR.

Main changes explained:

  • Routers , Controller and Model files created for this feature.

How to test:

  1. check into current branch
  2. do npm install and npm run build and npm start to run this PR locally
  3. Follow Steps on Frontend PR.

Screenshots or videos of changes:

Screen.Recording.2024-12-06.at.4.33.06.PM.mp4

@JatinAgrawal94 JatinAgrawal94 changed the title form is functioning well Jatin create different question sets Dec 7, 2024
@ppsahu
Copy link

ppsahu commented Dec 7, 2024

I have tried to review PR 2928, however, I am getting a reference error of "ReferenceError: jobsRouter is not defined" and the backend app is crashing out, so could not review the frontend PR. Below is the screenshot for reference. I have tried with npm start also, it gave the same error.

Screenshot 2024-12-07 at 4 21 00 AM

@mashpotato9
Copy link

I have tired to review and got the same jobsRouter is not defined ReferenceError as above

@JatinAgrawal94
Copy link
Contributor Author

I have tried to review PR 2928, however, I am getting a reference error of "ReferenceError: jobsRouter is not defined" and the backend app is crashing out, so could not review the frontend PR. Below is the screenshot for reference. I have tried with npm start also, it gave the same error.

Screenshot 2024-12-07 at 4 21 00 AM

This issue has been resolved now. Please review it again.

@JatinAgrawal94
Copy link
Contributor Author

I have tired to review and got the same jobsRouter is not defined ReferenceError as above

This issue has been resolve now. Please review the PR again.

@mashpotato9
Copy link

I have tired to review and got the same jobsRouter is not defined ReferenceError as above

Issue has been resolved, review left on #2928

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