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

feat: Whiteboard collaboration #16

Merged
merged 18 commits into from
Jun 6, 2024
Merged

feat: Whiteboard collaboration #16

merged 18 commits into from
Jun 6, 2024

Conversation

hweihwang
Copy link
Contributor

@hweihwang hweihwang commented May 22, 2024

Resolves #7

hweihwang and others added 6 commits May 16, 2024 17:18
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
Signed-off-by: grnd-alt <salimbelakkaf@outlook.de>
Signed-off-by: grnd-alt <salimbelakkaf@outlook.de>
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
hweihwang and others added 2 commits May 30, 2024 16:26
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
Signed-off-by: grnd-alt <salimbelakkaf@outlook.de>
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
hweihwang and others added 3 commits May 30, 2024 16:32
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
move helper functions to util.ts

Signed-off-by: grnd-alt <salimbelakkaf@outlook.de>
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
@hweihwang hweihwang force-pushed the feat/collab branch 3 times, most recently from 5a4d5e3 to 02bb523 Compare May 30, 2024 10:53
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
Signed-off-by: Hoang Pham <hoangmaths96@gmail.com>
Signed-off-by: grnd-alt <salimbelakkaf@outlook.de>
Signed-off-by: grnd-alt <salimbelakkaf@outlook.de>
@juliushaertl
Copy link
Member

Next steps as discussed with @hweihwang

  • Authentication based on JWT that is generated on the Nextcloud backend, handed over through the frontend to the nodejs server and then used to authenticate the get/save requests

@grnd-alt
Copy link
Member

grnd-alt commented Jun 6, 2024

@juliushaertl I have not been part of that discussion so there are surely reasons to do it that way. But I am wondering why we have to implement a new auth method for the node server. I though the node server would be gone soon as we initially planned to re-implement the web socket in php and could therefore use the nextcloud user-auth already. Are we going to keep the seperate node server?

@grnd-alt grnd-alt merged commit c79c286 into main Jun 6, 2024
1 check passed
@jancborchardt jancborchardt deleted the feat/collab branch July 8, 2024 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants