Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

V4.25.1 #1188

Merged
merged 5 commits into from
Jan 11, 2024
Merged

V4.25.1 #1188

merged 5 commits into from
Jan 11, 2024

Conversation

Cabecinha84
Copy link
Member

  • Fix syncthing where folders were not being updated if they were already present on syncthing, the problem is that the devices where that folder are supposed to connect are in the folders configuration, so if devices change, the folder would stop to sync with the new devices and so stop to sync if all devices from the first time the folder was created no longer are running the app or have a new device ID.

This version is the one that will be enforced instead of 4.25.0 and enforcement will happen on january 26th.
It's very important that node operators update as soon as possible to this new version.

@Cabecinha84 Cabecinha84 requested a review from TheTrunk January 11, 2024 13:35
Copy link

codecov bot commented Jan 11, 2024

Codecov Report

Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (44a89a0) 50.67% compared to head (52a50e0) 50.57%.
Report is 43 commits behind head on master.

Files Patch % Lines
ZelBack/src/services/appsService.js 0.00% 14 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1188      +/-   ##
==========================================
- Coverage   50.67%   50.57%   -0.10%     
==========================================
  Files          43       43              
  Lines       14101    14113      +12     
==========================================
- Hits         7145     7137       -8     
- Misses       6956     6976      +20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@TheTrunk TheTrunk merged commit d0d8dcd into master Jan 11, 2024
5 of 7 checks passed
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