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

OpenLP: update to 2.9.4. #36014

Merged
merged 5 commits into from Mar 7, 2022
Merged

Conversation

TinfoilSubmarine
Copy link
Contributor

Testing the changes

  • I tested the changes in this PR: briefly

Current version in repo is broken due to what I'm suspecting are just deprecation errors, and is also very old (March 31, 2017). This version is technically an RC for 3.0 but since it works and the current version in repo doesn't, I figured I would open a PR.

@paper42
Copy link
Member

paper42 commented Mar 7, 2022

@Chocimier

@TinfoilSubmarine
Copy link
Contributor Author

I pulled a patch that was added to the project after this tag to fix one of the errors, but the other one is still broken on the latest commit, so I just deselected it for now.

@Chocimier
Copy link
Member

Thanks, I do not use it daily, so didn't noticed it was broken by some pyqt(?) update.

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.

None yet

3 participants