-
Notifications
You must be signed in to change notification settings - Fork 26
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
Remove: Old app #14
Remove: Old app #14
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.
Perfection! Advancing through the open source world!🎊🎊 :) Please be patient while our maintainers and reviewers check the code.
@Akanchha112 please review this PR; since there were 2 different apps.,I deleted the old one. If this PR is merged, it will help others to make contribution in the updated one instead of the old one. |
@i-am-SnehaChauhan Meanwhile, @bj-jiwrajka there are still conflicts in the index.html and style.css, kindly look into it |
Alright! I've fixed the merge conflict, @i-am-SnehaChauhan can you please check now? |
@Akanchha112 I've fixed the merge conflict after your recent PR merge, can you please check now? |
Give me a little bit of time. I will review it and merge it. |
There were two apps in the same repository: one was the main one, and the other was within the "updated quiz app" folder. This PR removes the old app and keeps the updated one fixing minor bugs
@i-am-SnehaChauhan @Nitya-Pasrija please review this PR and merge it asap