kolibri
kolibri copied to clipboard
With 2.7 dropped, use shutil disk_usage function universally.
Summary
Removes platform specific behaviour. Uses shutil disk_usage which has support on Unix systems and Windows. This may well break Android, as it removes the Android specific code, but this will be rectified in the Android installer with this issue: https://github.com/learningequality/kolibri-installer-android/issues/211
References
Fixes the Kolibri side of https://github.com/learningequality/kolibri-installer-android/issues/211
Reviewer guidance
Windows and Unix systems should show free space appropriately for content imports.
Opening as a draft to first confirm the impact on the Android installer.
Testing checklist
- [ ] Contributor has fully tested the PR manually
- [ ] If there are any front-end changes, before/after screenshots are included
- [ ] Critical user journeys are covered by Gherkin stories
- [ ] Critical and brittle code paths are covered by unit tests
PR process
- [ ] PR has the correct target branch and milestone
- [ ] PR has 'needs review' or 'work-in-progress' label
- [ ] If PR is ready for review, a reviewer has been added. (Don't use 'Assignees')
- [ ] If this is an important user-facing change, PR or related issue has a 'changelog' label
- [ ] If this includes an internal dependency change, a link to the diff is provided
Reviewer checklist
- Automated test coverage is satisfactory
- PR is fully functional
- PR has been tested for accessibility regressions
- External dependency files were updated if necessary (
yarn
andpip
) - Documentation is updated
- Contributor is in AUTHORS.md
Build Artifacts
Asset type | Download link |
---|---|
PEX file | kolibri-.pex |
Windows Installer (EXE) | kolibri-0.16.2b2.dev0+git.1147.gc196a628-windows-setup-unsigned.exe |
Debian Package | kolibri_0.16.2b2.dev0+git.1147.gc196a628-0ubuntu1_all.deb |
Mac Installer (DMG) | kolibri-0.16.2b2.dev0+git.1147.gc196a628-0.4.2.dmg |
Android Package (APK) | kolibri-0.16.2b2.dev0+git.1147.gc196a628-0.1.3-debug.apk |
TAR file | kolibri-0.16.2b2.dev0+git.1147.gc196a628.tar.gz |
WHL file | kolibri-0.16.2b2.dev0+git.1147.gc196a628-py2.py3-none-any.whl |
Hi @rtibbles - the app force closes immediately after launch on all of my devices. Here are the logs, db and logcat from Android studio:
I guess that takes the uncertainty out of this!
This may well break Android
Looking at the logs, this is the same error you saw on my PR: https://github.com/learningequality/kolibri/pull/11974 - and doesn't seem to be directly related to this PR at all.
I will come back to this PR when the other one has been merged.
This might need a rebase, since https://github.com/learningequality/kolibri/pull/11974 has been merged... 🤔 (tested the APK today, the initial app force close after launch is still extant)
Have rebased on latest develop, so hopefully any errors should be purely its own!
Comment | Screenshhot |
---|---|
This latest APK installs and goes through the setup wizard without issues 🙂 Any specific reviewer guidance for this PR? |
The only possible source of breakage would be the free space being misreported during resource import.
Hi @rtibbles - no issues observed during resource import on the Android app and the 'Free disk space' value gets updated correctly.