Skip to content

Only normalize data in public shares and not crud - #33941

Merged
CarlSchwan merged 1 commit into
masterfrom
fix/public-sharing
Sep 15, 2022
Merged

Only normalize data in public shares and not crud#33941
CarlSchwan merged 1 commit into
masterfrom
fix/public-sharing

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

No description provided.

@CarlSchwanCarlSchwan added this to the Nextcloud 25 milestone Sep 7, 2022
@CarlSchwan
CarlSchwan requested review from a team and artongeSeptember 7, 2022 11:53
@CarlSchwanCarlSchwan self-assigned this Sep 7, 2022
@CarlSchwan
CarlSchwan requested review from PVince81, blizzz and come-nc and removed request for a teamSeptember 7, 2022 11:53

@artongeartonge left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Make sense

I can't find any occurrence of publicUpload in our code base, maybe we should deprecate it someday.

@blizzzblizzz mentioned this pull request Sep 7, 2022
@blizzz

Copy link
Copy Markdown
Member

@CarlSchwan integration-sharing-v1 failed

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@blizzzblizzz mentioned this pull request Sep 9, 2022

@PVince81PVince81 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@PVince81

Copy link
Copy Markdown
Member

@CarlSchwan please double check the failing sharing tests to see if related

@skjnldsvskjnldsv mentioned this pull request Sep 15, 2022
@CarlSchwan

Copy link
Copy Markdown
MemberAuthor

@CarlSchwan please double check the failing sharing tests to see if related

Tests work locally, so this seems to be a race condition as I saw the share link acceptance tests fail in other PR recently (probably caused by the redesign)

@CarlSchwan
CarlSchwan merged commit 02d55eb into masterSep 15, 2022
@CarlSchwan
CarlSchwan deleted the fix/public-sharing branch September 15, 2022 14:43
@mejo-

Copy link
Copy Markdown
Member

@CarlSchwan are you ok with backporting this to stable24?

@PVince81

Copy link
Copy Markdown
Member

/backport to stable24

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@CarlSchwan@blizzz@PVince81@mejo-@artonge