Skip to content

Conversation

MrAntoine77
Copy link
Contributor

@MrAntoine77 MrAntoine77 commented Jul 7, 2025

This PR introduces two new optional fields to the login process, allowing the capture to stop automatically:
Capture d'écran 2025-07-07 102041

Setting either value to 0 disables the corresponding limit.
image

@wolfpld
Copy link
Owner

wolfpld commented Jul 7, 2025

The memory limit is a duplication of already existing functionality (about window -> global settings -> memory limit).

I don't like the amount of clutter this change introduces. It should be hidden by default.

@Lectem Lectem force-pushed the feature/time-memory-limit branch from a99fef8 to 40715f1 Compare July 30, 2025 13:56
@Lectem
Copy link
Collaborator

Lectem commented Jul 30, 2025

I reworked the whole PR based on @MrAntoine77's work.
This now only adds the connection duration limit.

I think it may be interesting to allow an option for quick edit of some settings (perhaps making the about windows a real window that the user can keep open would be enough?). But that's a question for another issue/PR I think now.

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.

3 participants