Skip to content

🐛(backend) create_for_owner: add accesses before saving doc content#2124

Open
Ash-Crow wants to merge 1 commit intomainfrom
sbl-create-doc-accesses-before-content
Open

🐛(backend) create_for_owner: add accesses before saving doc content#2124
Ash-Crow wants to merge 1 commit intomainfrom
sbl-create-doc-accesses-before-content

Conversation

@Ash-Crow
Copy link
Collaborator

Purpose

We add the User Accesses before saving content so the user is sure to have access to the the first version when creating a doc through create_for_owner (fixes #2123)

Proposal

  • Changed the order of operations in ServerCreateDocumentSerializer
  • Added a unit test

We add the User Accesses before saving content so the user is sure to
have access to the the first version when creating a doc through
create_for_owner (fixes #2123)
@Ash-Crow Ash-Crow force-pushed the sbl-create-doc-accesses-before-content branch from 92a89f1 to f4670cd Compare March 24, 2026 18:20
@Ash-Crow Ash-Crow requested a review from lunika March 24, 2026 18:20
lunika
lunika previously approved these changes Mar 25, 2026
@lunika lunika dismissed their stale review March 25, 2026 08:13

tests are failing

@lunika
Copy link
Member

lunika commented Mar 25, 2026

The tests are failing on the tests you added

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.

Create accesses before saving content for meet transcription

2 participants