Skip to content
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

Added Automatic Instagram Profile Pic Download #62

Merged
merged 4 commits into from
Oct 4, 2020

Conversation

hardikkhurana
Copy link
Contributor

@hardikkhurana hardikkhurana commented Oct 3, 2020

It can be used easily by just inputting the username of the account

Description

Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.

Fixes #(issue_no)

Replace issue_no with the issue number which is fixed in this PR

Have you read the Contributing Guidelines on Pull Requests?

  • Yes
  • No

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Documentation Update

Checklist:

  • My code follows the style guidelines(Clean Code) of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have created a helpful and easy to understand README.md
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

It can be used easily by just inputting the username of the account
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congratulations!! for making your first PR at Amazing-Python-Scripts, our mentors will review it soon.

@hardikkhurana
Copy link
Contributor Author

Thank you for giving me this chance I hope you like my work.

@avinashkranjan
Copy link
Owner

@hardikkhurana Kindly fix syntax error, Before making a PR check the script on your system once
ApplicationFrameHost_C1k7pWBX1e

And Few More things...

  • Add a Readme file.
  • Attach a SS which shows working of your script.

Copy link
Owner

@avinashkranjan avinashkranjan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make the Changes Mentioned

Updated and Fixed all the errors also added some comments
Added SS to prove this code is running..
@avinashkranjan avinashkranjan linked an issue Oct 3, 2020 that may be closed by this pull request
@hardikkhurana
Copy link
Contributor Author

I have solved all the errors and added SS also the readme file.

Thank You

@hardikkhurana
Copy link
Contributor Author

@avinashkranjan Sir, have you checked my new PR?or is there any problem from my side?

@avinashkranjan
Copy link
Owner

@hardikkhurana I'm still looking into it...
ApplicationFrameHost_81sGITn3qi
Your scripts run but does nothing..

@hardikkhurana
Copy link
Contributor Author

@avinashkranjan Sir A new folder will be created after you run the script in the same folder as the python script is stored in and in that folder there will be the profile pictures as i have also showed in the Screenshots

@avinashkranjan
Copy link
Owner

@hardikkhurana No new Folder created.. Here the proof
explorer_joJvTVxYIZ

@hardikkhurana
Copy link
Contributor Author

@avinashkranjan I understand sir and would like to share this screen recorded video
here I am using Anaconda and Spyder

https://www.dropbox.com/s/v1rdwhua91pi3bc/Insta%20profile%20pic.mp4?dl=0

this is the link to that dropbox video

Thank you

@avinashkranjan
Copy link
Owner

avinashkranjan commented Oct 3, 2020

@hardikkhurana I'm still looking into it...

@hardikkhurana I knw..I think it's a problem from my side.... That's why I said..I'm currently looking into it..! Once it gets resolved I'll merge your PR

Copy link
Owner

@avinashkranjan avinashkranjan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM...💯

@avinashkranjan avinashkranjan merged commit b036783 into avinashkranjan:master Oct 4, 2020
@avinashkranjan
Copy link
Owner

@all-contributors please add @hardikkhurana for code and documentation

@allcontributors
Copy link
Contributor

@avinashkranjan

I've put up a pull request to add @hardikkhurana! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Instagram Profile Pic Downloader
2 participants