Skip to content

Conversation

vivek-2000
Copy link
Contributor

@vivek-2000 vivek-2000 commented Mar 19, 2021

@avinashkranjan
Script will follow and send msg
Fixes: #450

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.

Copy link
Contributor

@kaustubhgupta kaustubhgupta left a comment

Choose a reason for hiding this comment

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

Remove the chrome driver, add the instructions in the readme to place latest chrome driver in the directory

@@ -0,0 +1,4 @@
What is the objective of the Script
Copy link
Contributor

Choose a reason for hiding this comment

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

Format the readme properly, The headings need to be levels, level 2(##)


users=['lit_perception']

USERNAME='enter username'
Copy link
Contributor

Choose a reason for hiding this comment

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

get this from user

from selenium.webdriver.common.action_chains import ActionChains


users=['lit_perception']
Copy link
Contributor

Choose a reason for hiding this comment

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

get users from user

@kaustubhgupta kaustubhgupta added the bug Something isn't working label Mar 20, 2021
@vivek-2000
Copy link
Contributor Author

@kaustubhgupta updated

@@ -0,0 +1,23 @@
# Level2
Copy link
Contributor

Choose a reason for hiding this comment

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

Here replace level 2 with script name

browser = webdriver.Chrome(ChromeDriverManager().install())
time.sleep(10)
users=[]
users.append(input("Enter Users Username Whom You Want to Follow and Send Msg "))
Copy link
Contributor

Choose a reason for hiding this comment

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

how this will take multiple users? It will take all the comma-separated users as strings.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kaustubhgupta updated

@vivek-2000
Copy link
Contributor Author

@kaustubhgupta updated change required

@@ -0,0 +1,23 @@
# instabot.py
What is the use:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
What is the use:
## What is the use:


5. Once done, it will Show the Msg.

Note:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Note:
## Note:

Selenium, getpass and time


BY Vivek Kumar Patel No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
BY Vivek Kumar Patel
## By Vivek Kumar Patel

@vivek-2000
Copy link
Contributor Author

@kaustubhgupta Now updated

@kaustubhgupta kaustubhgupta added next review needed Approved by some mentors, more approvals needed and removed bug Something isn't working labels Mar 22, 2021
@vivek-2000
Copy link
Contributor Author

@antrikshmisri @santushtisharma10 can u plz review

Copy link
Contributor

@antrikshmisri antrikshmisri left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏻

@santushtisharma10 santushtisharma10 added Approved PR Approved and Ready to Merge gssoc23 Issues created for/by the GirlScript Summer of Code'23 Participants level3 New features, Major bug fixing level2 Bug fixing, Adding small features and removed next review needed Approved by some mentors, more approvals needed level3 New features, Major bug fixing labels Mar 23, 2021
@avinashkranjan avinashkranjan merged commit 26aa992 into avinashkranjan:master Mar 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved PR Approved and Ready to Merge gssoc23 Issues created for/by the GirlScript Summer of Code'23 Participants level2 Bug fixing, Adding small features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Insta Bot
5 participants