-wa command working successfully now

[SinghHimadri]

Issue that this pull request solves

347

Closes: # (issue number)

347

Proposed changes

Brief description of what is fixed or changed Error removed command working successfully and changing about now. Also readme has been updated correctly.

Types of changes

Put an x in the boxes that apply

• [x] Bugfix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
• [x] Documentation update (Documentation content changed)
• [ ] Other (please describe):

Checklist

Put an x in the boxes that apply

• [ ] My code follows the style guidelines of this project
• [ ] I have performed a self-review of my own code
• [ ] I have commented my code, particularly in hard-to-understand areas
• [x] I have made corresponding changes to the documentation
• [x] My changes generate no new warnings

Screenshots

Please attach the screenshots of the changes made in case of change in user interface

Other information

Any other information that is important to this pull request

[SinghHimadri]

@rpotter12 Please review my PR.

[SinghHimadri]

@SinghHimadri This software is a pypi package. So when a user install it from the command pip install wplay, the user will not able to paste the key in the script. The changes which you have made are only for those users who are running it from source code. :)

The file which is already present requires API key but even after adding API Key the code is showing error due to os.getenv() function. I have just removed that error.Web scrapping is not possible for this website as it requires captcha.Only thing that can be done(according to me) is making a list of eg: 10 keys and randomly placing it for the user. The current file doesn't work I have just removed that error.

[rpotter12]

@SinghHimadri May be we can ask user to enter api key in terminal for now. What do you say??

[priyansh19]

We can do one thing .. add a .config file and when the user runs the command the file will be checked if the API keys are not present then we will displays the steps for adding them 😅

[priyansh19]

Or we can directly take input from CLI also but it will be a bit insecure way of doing so

[SinghHimadri]

@rpotter12 I have done the changes please see.

[SinghHimadri]

@rpotter12 is their some issue left in this PR? I have done the changes you asked.