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

Delete Resharing from the code base #10771

Closed
7 of 9 tasks
micbar opened this issue Apr 12, 2024 · 9 comments · Fixed by #10809
Closed
7 of 9 tasks

Delete Resharing from the code base #10771

micbar opened this issue Apr 12, 2024 · 9 comments · Fixed by #10809
Assignees
Labels
Type:Story User Story

Comments

@micbar
Copy link
Contributor

micbar commented Apr 12, 2024

resharing feature is not supported anymore: https://github.com/owncloud/docs-ocis/pull/739/files#diff-04cd79e526c8a7f0162d8fc5bc5fa8c501fe9e5899024268837c05a74349b26cR148-R149

Web

  • Check that there are no leftover tests or test-steps that include resharing
  • Remove the resharing capability

cc @kobergj @saw-jan

@JammingBen
Copy link
Contributor

We already removed all the functionality regarding resharing. The only remaining things I can think of are:

  • Check that there are no leftover tests or test-steps that include resharing
  • Remove the resharing capability (what's the status on the server side here?)

Am I missing something?

@micbar
Copy link
Contributor Author

micbar commented Apr 12, 2024

Then we can close it :-)

@JammingBen
Copy link
Contributor

Okay, then let's leave it open for the remaining two tasks, I'll add them to the list above.

There is one thing not checked: default web client with resharing=false, does it need work on the server side?

@kulmann
Copy link
Member

kulmann commented Apr 12, 2024

Web doesn't need the resharing capability anymore then, but it needs to stay server side as long as other clients rely on it. Needs to be checked.

@kobergj
Copy link
Contributor

kobergj commented Apr 12, 2024

Remove the resharing capability (what's the status on the server side here?)

We couldn't remove it as web defaulted to true, we also need to check with other clients what their default is.

There is one thing not checked: default web client with resharing=false, does it need work on the server side?

Don't understand. The server doesn't know any more what resharing is.

@JammingBen
Copy link
Contributor

There is one thing not checked: default web client with resharing=false, does it need work on the server side?

Don't understand. The server doesn't know any more what resharing is.

Me neither 😄 I guess it can be checked then since both the Web client and the server don't know anything about resharing anymore.

@kulmann
Copy link
Member

kulmann commented Apr 12, 2024

There is one thing not checked: default web client with resharing=false, does it need work on the server side?

Don't understand. The server doesn't know any more what resharing is.

Me neither 😄 I guess it can be checked then since both the Web client and the server don't know anything about resharing anymore.

Well, server needs to announce resharing=false (hardcoded) until all clients are able to delete it (or default to false).

@kobergj
Copy link
Contributor

kobergj commented Apr 12, 2024

Since there is nothing to do here any more I close this. Feel free to reopen if you feel you still need this ticket

@kobergj kobergj closed this as completed Apr 12, 2024
@github-project-automation github-project-automation bot moved this from Backlog to Done in Infinite Scale Team Board Apr 12, 2024
@JammingBen
Copy link
Contributor

Let's keep the issue open so we keep track on the 2 remaining tasks in Web.

@JammingBen JammingBen reopened this Apr 12, 2024
@github-project-automation github-project-automation bot moved this from Done to In progress in Infinite Scale Team Board Apr 12, 2024
@JammingBen JammingBen moved this from In progress to Qualification in Infinite Scale Team Board Apr 12, 2024
@JammingBen JammingBen self-assigned this Apr 19, 2024
@JammingBen JammingBen moved this from Qualification to In progress in Infinite Scale Team Board Apr 19, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in Infinite Scale Team Board Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type:Story User Story
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants