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

Sharing fix #1691

Closed
wants to merge 6 commits into from
Closed

Sharing fix #1691

wants to merge 6 commits into from

Conversation

ishank011
Copy link
Contributor

No description provided.

@butonic butonic mentioned this pull request May 7, 2021
@labkode
Copy link
Member

labkode commented May 17, 2021

@ishank011 @butonic this change still doesn't make the CI test happy

@ishank011
Copy link
Contributor Author

ishank011 commented May 17, 2021

@pascalwengerter
Copy link
Contributor

@ishank011 what's the timeline for merging this? Just stumbled upon the fact that link-share navigation is broken in the current web due to the paths being too limited ;) I'll need those changes to close owncloud/web#5122

@ishank011
Copy link
Contributor Author

@pascalwengerter the tests need to be updated, else the CI would break. Please refer labkode#183 (comment). @butonic or @phil-davis will be able to give a better estimate

@ishank011
Copy link
Contributor Author

Closing this, continuing the discussion at labkode#183

@ishank011 ishank011 closed this May 25, 2021
@butonic butonic mentioned this pull request May 27, 2021
@ishank011 ishank011 deleted the sharing-fix branch October 27, 2021 11:51
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.

4 participants