-
Notifications
You must be signed in to change notification settings - Fork 51
Pranav2 #61
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
Pranav2 #61
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.
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.
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.
This issue is not mine. Opening notepad is someone else issue. Plz check.
My issue was to add Book Reader feature in PA.py which I have already done and made pull request which has no merge conflicts. PLz see to it and merge my pull request for book reader.
This issue is not mine. Opening notepad is someone else issue. Plz check. My issue was to add Book Reader feature in PA.py which I have already done and made pull request which has no merge conflicts. PLz see to it and merge my pull request for book reader.
Yes @Pranavv213 I know that this change was made in different PR but that PR is already merged. May be you haven't pulled the changes from the main branch that's why it is showing the previous code in your branch. Now, you can make the necessary changes, and then I will merge this PR. 👍
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.
@Pranavv213 are you working on this??
Book Reader changes made