Skip to content

[RELEASE] 4.8.2#4903

Open
joragua wants to merge 4 commits into
masterfrom
release/4.8.2
Open

[RELEASE] 4.8.2#4903
joragua wants to merge 4 commits into
masterfrom
release/4.8.2

Conversation

@joragua

@joragua joragua commented Jun 26, 2026

Copy link
Copy Markdown
Member

Related Issues

App: #4890


QA

Reports:

@joragua joragua self-assigned this Jun 26, 2026
@joragua joragua linked an issue Jun 26, 2026 that may be closed by this pull request
24 tasks
Signed-off-by: Jorge Aguado Recio <jaguado@izertis.com>
joragua and others added 2 commits June 26, 2026 10:27
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>
@joragua joragua marked this pull request as ready for review June 26, 2026 08:50
@joragua joragua requested a review from jesmrec June 26, 2026 08:50

@jesmrec jesmrec left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🆗

@jesmrec

jesmrec commented Jun 26, 2026

Copy link
Copy Markdown
Member

Moving to QA

@jesmrec jesmrec changed the title build: update versionCode and versionName for 4.8.2 [RELEASE] 4.8.2 Jun 26, 2026
@jesmrec

jesmrec commented Jun 26, 2026

Copy link
Copy Markdown
Member

(1) [FIXED]

  1. Select to copy or move a file

Current: When target location is submitted, the app crashes

Expected: no crash

Xiaomi Redmi 13, Android 15
Pixel 7, Android 16
Samsung Galaxy Tab A8, Android 14

04dc853ff

Signed-off-by: Jorge Aguado Recio <jaguado@izertis.com>
@joragua

joragua commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

(1) should be fixed in the commit 8d78aaf

@jesmrec

jesmrec commented Jun 26, 2026

Copy link
Copy Markdown
Member

(2)

Snackbar notifications overlaps the bottom bar. That seems to be related with infinite edges

Xiaomi Redmi 13, Android 15
Pixel 7, Android 16
8d78aaf028

@DeepDiver1975 DeepDiver1975 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.md Summary + 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.

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.

[RELEASE] 4.8.2

3 participants