Skip to content

Conversation

@frozenbanana
Copy link
Contributor

Pull Request Notes

PR #150: Docker updates for NC 32

Title: Docker updates for NC 32 (#150)

Description:
This PR updates the Docker configuration to support Nextcloud 32 and includes several infrastructure improvements.

Changes:

  • Docker Configuration Updates:

    • Bumped Docker setup to support Nextcloud 32
    • Fixed PostgreSQL version to 18 Alpine for better compatibility
    • Updated stage dependency services configuration
    • Updated default network IP configuration
  • Deployment Script Updates:

    • Updated deploy script version in cli/staging/deploy.sh
    • Improved deployment process for NC 32 compatibility
  • Dependencies:

    • Updated vendor-bin/sentry/composer.lock for better dependency management

Files Modified:

  • cli/staging/deploy.sh
  • docker-compose.stage.yml
  • docker-compose.yml
  • vendor-bin/sentry/composer.lock

Impact:

  • Enables support for Nextcloud 32
  • Improves Docker environment stability
  • Updates deployment process for better reliability

PR #151: Fix parent call on Traits

Title: fix: avoid parent call on Traits (#151)

Description:
This PR fixes a critical issue in the Telemetry class where parent calls were being made incorrectly on Trait classes, which could cause runtime errors.

Changes:

  • Bug Fix:

    • Fixed parent call issue in Trait classes in lib/AppInfo/Telemetry.php
    • Avoided calling parent methods on Trait classes which is not supported in PHP
  • Code Quality:

    • Applied code style fixes to maintain consistency
    • Improved code structure and readability

Files Modified:

  • lib/AppInfo/Telemetry.php

Impact:

  • Fixes potential runtime errors related to Trait usage
  • Improves code stability and reliability
  • Maintains proper PHP coding standards

PR #152: Fix/NC32 compatibility

Title: Fix/nc32 compatibility #1736 (#152)

Description:
This PR addresses Nextcloud 32 compatibility issues, particularly focusing on UI elements and overlay functionality.

Changes:

  • UI Fixes:

    • Fixed icon SVG rendering issues on Nextcloud 32
    • Updated CSS styling in css/app.css for better NC32 compatibility
    • Fixed button functionality and display issues
  • Overlay Improvements:

    • Updated overlay functionality in src/app/files/actions/overlay.js
    • Improved compatibility with Nextcloud 32's file management system
  • Documentation:

    • Updated README.md with latest compatibility information
    • Updated appinfo/info.xml with proper version and dependency information
  • Assets:

    • Updated logo file img/hejbit-logo-padded.svg for better rendering

Files Modified:

  • README.md
  • appinfo/info.xml
  • css/app.css
  • img/hejbit-logo-padded.svg
  • src/app/files/actions/overlay.js

Impact:

  • Ensures full compatibility with Nextcloud 32
  • Fixes UI rendering issues
  • Improves user experience on latest Nextcloud versions
  • Maintains backward compatibility with NC 30 and 31

Summary

These three PRs collectively:

  1. Enable Nextcloud 32 support through Docker and infrastructure updates
  2. Fix critical runtime issues in the Telemetry system
  3. Resolve UI compatibility problems specific to Nextcloud 32

All changes maintain backward compatibility with Nextcloud versions 30 and 31 while adding full support for version 32.

@frozenbanana frozenbanana merged commit 5d84dcf into main Oct 27, 2025
1 check passed
@frozenbanana frozenbanana deleted the release_20251027 branch October 27, 2025 13:15
frozenbanana added a commit that referenced this pull request Oct 27, 2025
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.

2 participants