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

[PRD-613] fix: Handle shared notes #2621

Merged
merged 2 commits into from
Nov 19, 2024
Merged

[PRD-613] fix: Handle shared notes #2621

merged 2 commits into from
Nov 19, 2024

Conversation

paultranvan
Copy link
Contributor

When a note is coming from a Cozy sharing, a special URL must be built, including the sharing instance URL, a sharecode and a the note ID on this instance.
The client does not have these information, so the stack is required to build the opening URL. This is directly handled by Drive, so when the user wants to open a shared note, we first need to redirect on Drive, that will redirect on Notes.

See https://github.com/cozy/cozy-drive/pull/3244/files

@paultranvan paultranvan requested a review from acezard as a code owner November 19, 2024 09:02
@paultranvan paultranvan requested review from Ldoppea and cballevre and removed request for acezard November 19, 2024 09:02
@paultranvan paultranvan changed the title fix: Handle shared notes [PRD-613] fix: Handle shared notes Nov 19, 2024
@paultranvan paultranvan force-pushed the fix-sharing-link branch 3 times, most recently from ccbbd62 to b46bd48 Compare November 19, 2024 09:34
paultranvan and others added 2 commits November 19, 2024 10:54
When a note is coming from a Cozy sharing, a special URL must be built,
including the sharing instance URL, a sharecode and a the note ID on
this instance.
The client does not have these information, so the stack is required to
build the opening URL. This is directly handled by Drive, so when the
user wants to open a shared note, we first need to redirect on Drive,
that will redirect on Notes.

See https://github.com/cozy/cozy-drive/pull/3244/files
Copy link
Contributor

@cballevre cballevre left a comment

Choose a reason for hiding this comment

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

LGTM 👍 We'll just have to bear in mind that version 1.74.2 has to be in production to be able to use it in production

@paultranvan paultranvan merged commit ec8014c into master Nov 19, 2024
2 checks passed
@paultranvan paultranvan deleted the fix-sharing-link branch November 19, 2024 15:40
paultranvan added a commit to cozy/cozy-web-data-proxy that referenced this pull request Nov 20, 2024
paultranvan added a commit to cozy/cozy-web-data-proxy that referenced this pull request Nov 20, 2024
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.

3 participants