-
Notifications
You must be signed in to change notification settings - Fork 46
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
Update login.tsx #1045
Update login.tsx #1045
Conversation
@nipunh is attempting to deploy a commit to the Chayn Team on Vercel. A member of the Team first needs to authorize it. |
Hi @nipunh thank you for your PR! Everything looks good. Just one thing, we need to run Cypress tests. |
I was facing some problems with the backend setup and database seeding |
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.
@nipunh understood. I ran the tests this time. Next time, let us know the problems you are facing so we can set you up.
The dependency array on line 92 also needs updated. Ill fix that then approve after tests pass.
Test summaryRun details
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud |
Issue link / number:
#1044
What changes did you make?
Update the check requested in issue
Why did you make the changes?
if (partnerAdmin?.id) to if (partnerAdmin?.active)
Did you run tests?
No
@kyleecodes