That PR was merged into PR #36573 and #36573 is still waiting to be merged (getting close). It has its own changelog entry https://github.com/owncloud/core/pull/36573/files#diff-84a69c0d07819b15fa72450563c4a156

Do we also need this changelog entry?

If so, then do we cherry-pick it into #36573 rather than merging it to master now?

@phil-davis This changelog entry is needed IMO because it describes the changes in the default behavior.

Feel free to cherry-pick and close here.

@micbar #36573 is about to finish and pass CI. If someone could review and approve it then it can be merged.
Then we can just leave CI running on this PR and also merge it in 1 hours time.

Codecov Report

Merging #36617 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #36617   +/-   ##
=========================================
  Coverage     64.68%   64.68%           
  Complexity    19071    19071           
=========================================
  Files          1268     1268           
  Lines         74522    74522           
  Branches       1312     1312           
=========================================
  Hits          48207    48207           
  Misses        25930    25930           
  Partials        385      385

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 41b46f7...b51a3a3. Read the comment docs.

Feel free to merge this when #36573 is merged.