-
Notifications
You must be signed in to change notification settings - Fork 0
Unclear message on incorrect credentials #280
base: master
Are you sure you want to change the base?
Unclear message on incorrect credentials #280
Conversation
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.
- The submodules, Pong-Game, VisualGit_SE701_2019_3, and dummy, should be removed from this pull request.
- The image pexels-photo-414612 does not seem to used, this should be removed.
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.
The submodules, Pong-Game, VisualGit_SE701_2019_3, and dummy, should be removed from this pull request.
The image pexels-photo-414612 does not seem to used, this should be removed.
theres some changes that dont seem to do anything, could you fix this, thanks.
outdated comments - resolved by latest commit
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.
Your code looks good, but it seems that you have merge conflicts with master, I am unable to test your code, due to these conflicts, could you correct this as well, thanks
@iamkunalb is this working on your system, for some reason when I test it I just get a bunch of errors and nothing loads also the transcript compiler is getting some compiler issues from the file you have made modifications to, I'm not sure if this is causing the page to not load |
Fixed unclear pop up message on incorrect credentials
Test:
or