Skip to content

fix: run update check in updatenotification app only when internet co…#41262

Merged
DeepDiver1975 merged 1 commit intomasterfrom
fix/update-notifications-only-with-internet-connection
Jun 7, 2024
Merged

fix: run update check in updatenotification app only when internet co…#41262
DeepDiver1975 merged 1 commit intomasterfrom
fix/update-notifications-only-with-internet-connection

Conversation

@DeepDiver1975
Copy link
Copy Markdown
Member

@DeepDiver1975 DeepDiver1975 commented Jun 6, 2024

…nnection is available

Description

In case an owncloud instance is not connected to the internet there is no need in polling for updates in the update notification app.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE

@update-docs
Copy link
Copy Markdown

update-docs bot commented Jun 6, 2024

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@DeepDiver1975 DeepDiver1975 marked this pull request as draft June 6, 2024 11:24
@DeepDiver1975 DeepDiver1975 force-pushed the fix/update-notifications-only-with-internet-connection branch from 253637d to 83909ad Compare June 6, 2024 12:45
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Jun 6, 2024

@DeepDiver1975 DeepDiver1975 marked this pull request as ready for review June 6, 2024 14:49
@pako81 pako81 self-requested a review June 7, 2024 06:09
@DeepDiver1975 DeepDiver1975 merged commit 07ad623 into master Jun 7, 2024
@delete-merged-branch delete-merged-branch bot deleted the fix/update-notifications-only-with-internet-connection branch June 7, 2024 06:17
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