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

Feat custom propagator #2440

Closed

Conversation

gbrousse-recia
Copy link

@gbrousse-recia gbrousse-recia commented Jun 21, 2023

Description

A custom Propagator to fix files updates propagation in nested groupfolders.
Based on branch stable25.
Tested on Nextcloud 25.0.6.

Related Issue

maybe #652 and #1787

Motivation and Context

We had synchronization issues between the server and the client when the files were in nested groupfolders. After some research, we noticed that the propagation was not done correctly (groupfolders etags unchanged in the filecache table).

@gbrousse-recia gbrousse-recia force-pushed the feat_propagator branch 3 times, most recently from 8cac209 to 6cda676 Compare June 22, 2023 11:52
Signed-off-by: gbrousse-recia <[email protected]>
Signed-off-by: gbrousse-recia <[email protected]>
@gbrousse-recia gbrousse-recia marked this pull request as ready for review June 22, 2023 14:31
@kesselb
Copy link
Contributor

kesselb commented Jun 22, 2023

Thanks for your pull request 👍

Our usual workflow is to submit a change to branch master first and back port to stable (the latter is done by a bot).

@kesselb kesselb added bug 2. developing Items that are currently under development 3. to review Items that need to be reviewed and removed 2. developing Items that are currently under development labels Jun 22, 2023
@juliusknorr
Copy link
Member

Thanks a lot @gbrousse-recia this looks promising, but as @kesselb asked, could you please open a pull request for this towards the master branch?

@tacruc
Copy link

tacruc commented Aug 30, 2023

@gbrousse-recia Do you mind if I cherry pick this and open a new PR to the master branch?

@gbrousse-recia
Copy link
Author

@gbrousse-recia Do you mind if I cherry pick this and open a new PR to the master branch?

@tacruc, Yes, no problem, if you think it is revelant.

@tacruc tacruc mentioned this pull request Aug 31, 2023
@blizzz
Copy link
Member

blizzz commented Nov 22, 2023

closing in favor of #2533, also 25 is EOL

@blizzz blizzz closed this Nov 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Items that need to be reviewed bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants