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 Onboarding for HR and roles #545

Open
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

DeepanshuMishraa
Copy link
Contributor

@DeepanshuMishraa DeepanshuMishraa commented Oct 23, 2024

closes #520

  • Added Onboarding Flow for HRs to setup their company while signup
  • Normal users wont have the need to setup company info
  • Gives HR roles to HR while User role to job-seekers

HR FLOW:

100xJobs.Zen.Browser.2024-10-24.01-10-33.mp4

User Flow:

100xJobs.Zen.Browser.2024-10-24.01-09-52.mp4

@VineeTagarwaL-code
Copy link
Collaborator

we need a company logo too, can you add that ?

@DeepanshuMishraa
Copy link
Contributor Author

Yes ser on it

@VineeTagarwaL-code
Copy link
Collaborator

okay do that asap, and show me the full flow

  • for hr
  • for user

@DeepanshuMishraa
Copy link
Contributor Author

@VineeTagarwaL-code take a look at it

@DeepanshuMishraa
Copy link
Contributor Author

@VineeTagarwaL-code do I need to change something In here?

@VineeTagarwaL-code
Copy link
Collaborator

give a video ref of current changes

@DeepanshuMishraa
Copy link
Contributor Author

give a video ref of current changes

I have update the PR description and added the video ref for hr and user flow . Should I give the video ref for something else

@DeepanshuMishraa
Copy link
Contributor Author

@VineeTagarwaL-code is this much video ref as shown in the PR description fine or should i give ref for more things?

@VineeTagarwaL-code
Copy link
Collaborator

code has a lot of bugs, pointing them all out right now

src/actions/job.action.ts Outdated Show resolved Hide resolved
src/actions/job.action.ts Outdated Show resolved Hide resolved
prisma/seed.ts Outdated Show resolved Hide resolved
src/components/auth/signup.tsx Outdated Show resolved Hide resolved
src/actions/auth.actions.ts Outdated Show resolved Hide resolved
src/actions/job.action.ts Show resolved Hide resolved
src/components/auth/signup.tsx Show resolved Hide resolved
src/lib/validators/auth.validator.ts Outdated Show resolved Hide resolved
src/lib/validators/auth.validator.ts Outdated Show resolved Hide resolved
@DeepanshuMishraa
Copy link
Contributor Author

@VineeTagarwaL-code got it sir , yes i see i messed up a lot will try to fix it asap

@DeepanshuMishraa
Copy link
Contributor Author

@VineeTagarwaL-code fixed all the things that you pointed ser, pls look into it and let me know if i missed something or if something else is not right

@DeepanshuMishraa
Copy link
Contributor Author

@VineeTagarwaL-code

@DeepanshuMishraa
Copy link
Contributor Author

@VineeTagarwaL-code any review upon this?

@VineeTagarwaL-code
Copy link
Collaborator

fix conflicts

@DeepanshuMishraa
Copy link
Contributor Author

@VineeTagarwaL-code done sir

@DeepanshuMishraa
Copy link
Contributor Author

@VineeTagarwaL-code is this alright?

@DeepanshuMishraa
Copy link
Contributor Author

@VineeTagarwaL-code any updates ?

@VineeTagarwaL-code
Copy link
Collaborator

Sorry for not replying, had to merge other prs and get them sorted out !

i will review this tomorrow morning ( 1st task )

@Smnthjm08
Copy link

@VineeTagarwaL-code is this feature still open??

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.

🚀 Feature: Onboarding and role selection for new comers, adding company onboarding flow
3 participants