Skip to content

[stable31] Fix permission change in text doc - #8748

Merged
max-nextcloud merged 3 commits into
stable31from
backport/8746/stable31
Aug 5, 2026
Merged

[stable31] Fix permission change in text doc#8748
max-nextcloud merged 3 commits into
stable31from
backport/8746/stable31

Conversation

@backportbot

@backportbotbackportbotBot commented Jun 16, 2026•
edited by samin-z
Loading

Copy link
Copy Markdown

Backport of #8746

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@silverkszlo@backportbot
test(ApiService): test save with NotPermittedException
d2df64d
Signed-off-by: silver <s.szmajduch@posteo.de>
@silverkszlo@samin-z
run cs:fix
c142d3a
run cs:fix
Signed-off-by: silver <s.szmajduch@posteo.de>
@samin-z
samin-zforce-pushed the backport/8746/stable31 branch 3 times, most recently from 4932000 to 4304180CompareJune 17, 2026 16:50
@samin-z
samin-z marked this pull request as ready for review June 17, 2026 16:50
silverkszlo
silverkszlo reviewed Jun 18, 2026
View reviewed changes
Comment threadsrc/services/PollingBackend.js
* @type {number}
*/
const FETCH_INTERVAL_READ_ONLY = 30000
const FETCH_INTERVAL_READ_ONLY = 5000

@silverkszlosilverkszloJun 18, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if we want to merge this PR, we should set this back to 30000

@samin-z
manual update for missing files
240854c
Signed-off-by: samin-z <samin.zavarkesh@gmail.com>
@samin-z
samin-zforce-pushed the backport/8746/stable31 branch from 4304180 to 240854cCompareJuly 1, 2026 15:10
@samin-z

samin-z commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

@silverkszlo@max-nextcloud stable31 is also now aligned with other versions, no custom changes, another round of review is highly appreciated.

@silverkszlo

silverkszlo commented Jul 2, 2026•
edited
Loading

Copy link
Copy Markdown
Collaborator

Thank you for tackling that! I tested it and it works nicely. I just commented some small things. There is one unrelated bug:

The browser console throws a lot of these errors:
Screenshot from 2026-07-02 09-13-17

That can be fixed by adding the useIsRichWorkspaceMixin to ReadonlyBar.vue. That actually only applies to stable31, because later stables and main then use the useEditorFlags() composables instead of the provide/inject mixin pattern. I'll fix that in a separate PR.

@github-project-automationgithub-project-automationBot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity teamJul 15, 2026
@max-nextcloudmax-nextcloud moved this from 🧭 Planning evaluation (don't pick) to 👀 In review in 📝 Productivity teamJul 15, 2026
@max-nextcloud
max-nextcloud merged commit bc9190f into stable31Aug 5, 2026
68 checks passed
@max-nextcloud
max-nextcloud deleted the backport/8746/stable31 branch August 5, 2026 09:16
@github-project-automationgithub-project-automationBot moved this from 👀 In review to ☑️ Done in 📝 Productivity teamAug 5, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@silverkszlosilverkszlosilverkszlo left review comments
@benjaminfruehbenjaminfruehAwaiting requested review from benjaminfrueh
@max-nextcloudmax-nextcloudAwaiting requested review from max-nextcloudmax-nextcloud is a code owner
@mejo-mejo-Awaiting requested review from mejo-mejo- is a code owner

Assignees

@silverkszlosilverkszlo

Labels

3. to review

Projects

📝 Productivity team
Status: ☑️ Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@samin-z@silverkszlo@max-nextcloud