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

Add peerflix-style interactive select mode #234

Merged
merged 5 commits into from
Jun 2, 2022

Conversation

alexose
Copy link
Contributor

@alexose alexose commented Apr 6, 2022

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[ ] Bug fix
[X] New feature
[ ] Other, please explain:

What changes did you make? (Give an overview)

Added a peerflix-style interactive select mode.

Which issue (if any) does this pull request address?

#130

Is there anything you'd like reviewers to focus on?

Hopefully I got the formatting right!

@welcome
Copy link

welcome bot commented Apr 6, 2022

🙌 Thanks for opening this pull request! You're awesome.

@alexose
Copy link
Contributor Author

alexose commented May 20, 2022

Ping!

@DiegoRBaquero
Copy link
Member

Can you add a video/gif of it working? Also, are there any scenarios were the await might throw an error? if that's the case, would be great to handle it

@alexose
Copy link
Contributor Author

alexose commented May 31, 2022

Here's a gif of it in action:

webtorrent

@alexose
Copy link
Contributor Author

alexose commented May 31, 2022

Per inquisitor's README.md, it can indeed throw errors if the terminal isn't capable of rendering the menu for some reason. The above commit should handle this case (and anything else that might result in an error).

@DiegoRBaquero DiegoRBaquero merged commit 4540a0a into webtorrent:master Jun 2, 2022
@welcome
Copy link

welcome bot commented Jun 2, 2022

🎉 Congrats on getting your first pull request landed!

webtorrent-bot pushed a commit that referenced this pull request Jun 2, 2022
# [4.1.0](v4.0.4...v4.1.0) (2022-06-02)

### Features

* add peerflix-style interactive select mode ([#234](#234)) ([4540a0a](4540a0a))
@webtorrent-bot
Copy link

🎉 This PR is included in version 4.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

3 participants