✨(frontend) Duplicate a doc #1078
Conversation
AntoLC
force-pushed
the
feature/duplicate
branch
7 times, most recently
from
e88db26 to
b90a091
Compare
AntoLC
force-pushed
the
feature/duplicate
branch
2 times, most recently
from
855ad40 to
4e36ce7
Compare
Yes we can |
|
Hey @AntoLC @rl-83 talked with @sampaccoud about this. |
By default we shouldn't copy the associated members and rights. Let's remove this from this PR. I've opened an issue to add the mentioned above |
AntoLC
force-pushed
the
feature/duplicate
branch
3 times, most recently
from
7cecebd to
a6d538b
Compare
| await editor.click(); | ||
| await editor.fill('Hello Duplicated World'); | ||
|
|
||
| await page.reload(); |
There was a problem hiding this comment.
This does not reflect what really happen when you work on a document. This reload I think trigger the save of the document content, this is why you have the same content at the end of this test
There was a problem hiding this comment.
It will now save if possible when duplication.
- if reader, it will not save, so 1mn could be lost with duplication if editor are writing in the same moment.
- if duplication is trigger from option document list, we cannot access the provider, so 1mn could be lost with duplication if editor are writing in the same moment.
We were maintaining two separate components for AGPL and MIT license exports. This commit consolidates the functionality into a single component that handles both licenses, simplifying the codebase and reducing duplication.
We can duplicate a document from the tool options.
The duplicate was also able for anonynous user if they can read it. We have to restrict it to at least reader authenticated otherwise no access will be created on the duplicated document.
Only admin or owner should be able to duplicate a document with existing accesses.
Purpose
Gives the possibility to duplicate a document.
Proposal
Demo
Demo.duplicate.doc.-.Docs.webm
Edge case
We save before duplication to ensure the latest state is duplicated.