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

Remove Limited Session #3545

Merged
merged 1 commit into from May 13, 2017
Merged

Conversation

@zonkmachine
Copy link
Member

zonkmachine commented May 8, 2017

As Limited Session is useful for precisely no one at all, I'm removing it. This is commit one from the #3032 rebased against stable-1.2 .

@zonkmachine
Copy link
Member Author

zonkmachine commented May 11, 2017

@tresf @Umcaruje This is good stuff. The Limited Session only really worked for me while I was actively working with the PR introducing it itself but it is of no value to keep.This is not a good substitute for multiple session recover files but just confuse the user. I will merge this is in a day or so if there are no objections.

@zonkmachine
Copy link
Member Author

zonkmachine commented May 13, 2017

No objections, merge. 🚜

@zonkmachine zonkmachine merged commit 290556e into LMMS:stable-1.2 May 13, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@zonkmachine zonkmachine deleted the zonkmachine:limitedSessionDropped2 branch May 13, 2017
@Umcaruje
Copy link
Member

Umcaruje commented May 13, 2017

👍

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

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.