[RELEASE] 4.8.2#4903
Conversation
Signed-off-by: Jorge Aguado Recio <jaguado@izertis.com>
Signed-off-by: Jorge Aguado Recio <jaguado@izertis.com>
Co-authored-by: owncloud-calens-bot[bot] <276982735+owncloud-calens-bot[bot]@users.noreply.github.com>
|
Moving to QA |
(1) [FIXED]
Current: When target location is submitted, the app crashes Expected: no crash Xiaomi Redmi 13, Android 15
|
Signed-off-by: Jorge Aguado Recio <jaguado@izertis.com>
|
(1) should be fixed in the commit 8d78aaf |
(2)Snackbar notifications overlaps the bottom bar. That seems to be related with infinite edges Xiaomi Redmi 13, Android 15 |
DeepDiver1975
left a comment
There was a problem hiding this comment.
Release review for 4.8.2 (maintainer triage, non-blocking — release PRs are owned by the release manager).
Version bump — consistent: versionCode 48000100 → 48000200 and versionName "4.8.1" → "4.8.2" in owncloudApp/build.gradle. Monotonic and matches the tag.
Changelog — well-formed (calens convention):
- TOC entry, section header, and intro line all updated unreleased → 4.8.2 (2026-06-30).
- Compare link correctly rewritten to
v4.8.1...v4.8.2. - Three fragments moved
changelog/unreleased/→changelog/4.8.2_2026-06-30/(4887 Bugfix: Infinite edges, 4897 Security: Redact cookie headers in logs, 4901 Bugfix: Remove useless debug manifest file) — each follows TEMPLATE (Type: Summary / blank / body / blank / URLs). - Generated
CHANGELOG.mdSummary + Details sections match the fragments. Generate Changelog check passes.
Code change — the fix: use content layout to show snackbar with action commit (drops the layoutId param across ActivityExt.kt, FileActivity.java, FileDisplayActivity.kt, always using android.R.id.content) is the QA-stabilization fix for the move/copy crash (report #1 in the PR body, marked FIXED). Folding a regression fix into the release branch during QA is fine. Minor note: it has no dedicated changelog fragment, but it's an in-cycle fix on top of the already-listed edge-to-edge feature (#4887), so no separate entry is required.
All CI green (Detekt, unit + instrumented data tests, CodeQL, validations) and already approved by @jesmrec. No stray/unrelated code. Looks good to ship from a maintainer standpoint.
Note: QA report #2 (snackbars overlapping bottom bar) is still unchecked in the PR description — leaving the merge decision to the release manager.
Related Issues
App: #4890
QA
Reports: