Skip to content

Allow specifying a default expiration date - #20587

Merged
rullzer merged 1 commit into
masterfrom
enh/allow_default_expiration_date
Apr 24, 2020
Merged

Allow specifying a default expiration date#20587
rullzer merged 1 commit into
masterfrom
enh/allow_default_expiration_date

Conversation

@rullzer

Copy link
Copy Markdown
Member

This overrides the max expiration date.

Signed-off-by: Roeland Jago Douma roeland@famdouma.nl

@nickvergessennickvergessen 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.

:-|

Comment threadtests/lib/Share20/ManagerTest.php Outdated
@rullzer
rullzerforce-pushed the enh/allow_default_expiration_date branch from 0fb2d10 to 809914bCompareApril 21, 2020 12:22
This was referenced Apr 21, 2020
@rullzerrullzer added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Apr 24, 2020
This overrides the max expiration date.
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer
rullzerforce-pushed the enh/allow_default_expiration_date branch from 809914b to f4b8623CompareApril 24, 2020 14:18
@rullzerrullzer added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Apr 24, 2020
@rullzer

Copy link
Copy Markdown
MemberAuthor

READY FOR REVIEW

@georgehrkegeorgehrke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 24, 2020
@rullzer
rullzer merged commit 652639b into masterApr 24, 2020
@rullzer
rullzer deleted the enh/allow_default_expiration_date branch April 24, 2020 15:08
@rullzer

Copy link
Copy Markdown
MemberAuthor

/backport to stable18

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable18 in #20635 with conflicts ⚠️

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

Labels

4. to releaseReady to be released and/or waiting for tests to finishenhancementfeature: sharing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@rullzer@nickvergessen@georgehrke@ChristophWurst