-
Notifications
You must be signed in to change notification settings - Fork 28
leaderboard solved issue #17 #57
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for computercodingtest ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
The route for leaderboard is changed from 'leader-board' to 'leaderboard'. hamburger and nabber's leader board button directs to different routes. The button in ham-burger menu is directing to route 'leaderboard' and the navbar in desktop view is directing to route 'leader-board'. Unable to access leaderboard in desktop mode. Please change 'leaderboard' to 'leader-board' in:
|
|
@adityarai0705 i have made PR but sir actually after implementing changes when i am trying to check them in web because of localhost i am unable to login can you suggest a way please |
|
@RITIK-CHAUDHRY If not, please refer to the dummy_env file in the server-side directory and the env file in the client-side directory to set all environment variables correctly. Once configured, try registering using your localhost setup for both the client and server. Let me know if you encounter any issues! |
@RITIK-CHAUDHRY |
|
@RITIK-CHAUDHRY , please create PR for branch 'version-1.2' instead of 'main'. |
|
@RITIK-CHAUDHRY , do you have any update? |
|
sir give me time till tommorow night i actually caught up with cold. I will make the PR of all my assigned ones tommorow |
|
@RITIK-CHAUDHRY, |
add leaderboard to the mobile phones resolved issue #17