Skip to content

Conversation

@Sumit112192
Copy link

…e code

Pull Request Label:

  • Bug
  • New feature
  • Enhancement
  • New component
  • Typo

Pull Request Checklist:

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same fix or component?
  • Have you run the tests locally with bats tests?
    I ran the tests locally, but the issues weren't related to my changes.

The Problem with ytview was that youtube had changed its HTML structure, but ytview was still expecting the old HTML structure. I tweaked the file ytview to make it work. Now, anyone can search for a topic and watch the video. Though, the channel part still needs to be re-implemented.

Previous Issue and Output:
Screenshot from 2023-07-11 19-37-11

Screenshot from 2023-07-11 19-37-32

Output after the change:
Screenshot from 2023-07-11 19-53-16
Screenshot from 2023-07-11 19-53-37

Also, initially, the command "search=$(printf '%s ' "$@")" would result in -s being added along with the string to be searched to the search variable, but now only the string to be searched is added. The only compromise is that the user has to enter the string in double or single quotes. And for that, I have changed the help menu.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant