Skip to content

Conversation

@DeepDiver1975
Copy link
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

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

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.

3 participants