Skip to content

[stable28] enh: Fix display default expire date, add tests & tiny refactors - #50765

Merged
AndyScherzinger merged 4 commits into
stable28from
backport/50693/stable28
Feb 12, 2025
Merged

[stable28] enh: Fix display default expire date, add tests & tiny refactors#50765
AndyScherzinger merged 4 commits into
stable28from
backport/50693/stable28

Conversation

@backportbot

@backportbotbackportbotBot commented Feb 11, 2025

Copy link
Copy Markdown

Backport of #50693

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
Since `ShareEntryLink` component is used to both create and display/list the share links,
we should only set default expiration date on `share.expireDate` when we know is a new share.
Otherwise, we overidding data from the backend.
Signed-off-by: nfebe <fenn25.fn@gmail.com>
@artongeartonge changed the title [stable28] [stable29] enh: Fix display default expire date, add tests & tiny refactors[stable28] enh: Fix display default expire date, add tests & tiny refactorsFeb 12, 2025
@nfebe
nfebeforce-pushed the backport/50693/stable28 branch from 8a86f05 to 6c9032cCompareFebruary 12, 2025 11:24
@nfebe
nfebe marked this pull request as ready for review February 12, 2025 11:24
@nfebe

Copy link
Copy Markdown
Contributor

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@AndyScherzinger
AndyScherzinger merged commit f47653b into stable28Feb 12, 2025
@AndyScherzinger
AndyScherzinger deleted the backport/50693/stable28 branch February 12, 2025 12:23
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsfeature: sharing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@nfebe@artonge@AndyScherzinger@nextcloud-command