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

Feat/company details management page #496

Closed

Conversation

Paribesh01
Copy link
Contributor

Screen.Recording.2024-10-11.at.12.04.17.PM.mov

@Paribesh01 Paribesh01 closed this Oct 11, 2024
@Paribesh01 Paribesh01 reopened this Oct 11, 2024
@VineeTagarwaL-code
Copy link
Collaborator

Wohooooo nice

@VineeTagarwaL-code
Copy link
Collaborator

will review and merge in this weekened ( tomorrow )

@CuriousCoder00
Copy link
Contributor

Is there anything else that needs to be added in it? or needs to be changed

@Paribesh01
Copy link
Contributor Author

There are some bugs, working on it

@VineeTagarwaL-code
Copy link
Collaborator

fix conflict

@Paribesh01
Copy link
Contributor Author

@VineeTagarwaL-code this should work

@VineeTagarwaL-code
Copy link
Collaborator

give me video ref of your current status

@CuriousCoder00
Copy link
Contributor

continuing from here in #524 as i cannot push my changes to your fork @Paribesh01

  • Fixed some bugs
  • added condition for rendering the company details page only after login
  • updated fetching of companies

@VineeTagarwaL-code please review the PR

@VineeTagarwaL-code
Copy link
Collaborator

will wait for the new ui, will ask you to make that here itself

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