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 shutdown_delay on the start() function parameter #529

Merged
merged 9 commits into from
Oct 19, 2021

Conversation

gaetan1903
Copy link
Contributor

timer configurable for Eel's shutdown detection mechanism, whereby when any websocket closes, it waits shutdown_delay seconds (default 1 second), and then checks if there are now any websocket connections. If not, then Eel closes. In case the user has closed the browser and wants to exit the program.

Try to fix #355 issues: Increase the value shutdown_delay higher to avoid stopping Eel.

gaetan1903 and others added 8 commits February 26, 2021 12:12
- Change default if time_shutdown is not correct int or float
- Accept string if is a digit
fix flake8 warning
remplace shortcut `f` not stable on python3.6 by format
@gaetan1903
Copy link
Contributor Author

how about it @ChrisKnott ?

Copy link
Collaborator

@ChrisKnott ChrisKnott left a comment

Choose a reason for hiding this comment

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

Simple change that I'm sure people will find useful

eel/__init__.py Outdated Show resolved Hide resolved
I think the syntax of this multi-line string was not correct
@yunyuyuan
Copy link

This pr has not been released,latest release is 0.14.0 at 2020,so shutdown_delay is not available yet.🥲

@samuelhwilliams
Copy link
Collaborator

samuelhwilliams commented Nov 8, 2022

@dstricks Chris and I never released this! 😱 Could be a good candidate for you to prepare a 0.15.0 release. You'll need to add entries to the changelog, bump the version number, ands prepare a version bump PR (similar to #382).

If you want to put that up, I'm happy to approve it. After it's in you can tag the release from here: https://github.com/python-eel/Eel/releases/new and it should automatically get packaged up and released.

@dstricks
Copy link
Contributor

dstricks commented Nov 8, 2022

@samuelhwilliams Sounds good. Will be the next thing I work on...

@dstricks dstricks mentioned this pull request Nov 9, 2022
@dstricks
Copy link
Contributor

v0.15.0 has been released and is available on PyPI.org

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.

ERR_CONNECTION_REFUSED
5 participants