Project

General

Profile

Actions

Feature #66948

closed

Make workspaces.previewLinkTTLHours setting part of the WS itself

Added by Patrick Gaumond almost 9 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Workspaces
Target version:
-
Start date:
2015-05-13
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Right now the preview links lifetime in WS are related to User TSconfig. In an environment with many WS, some may need different duration/TTL. Make it related to user makes no sense in such installation.

If someone needs TTL of 2 days, 7 days and 30 days with Preview links genrated by the same editor, how could this be done with conditions or anything at the moment? Creating "dummy users with different TTL" is not quite elegant...

3 potential solutions :

  1. A quick fix could be to add PAGE TSconfig support for workspaces.previewLinkTTLHours.
  2. Another way to solve the problem could be to let editors in BE to choose from a dropdown menu or enter a duration of the preview link then save it into sys_preview with that timestamp.
  3. An even more satisfactory solution would be to store the TTL value in the WS itself.
Actions #1

Updated by Oliver Hader over 8 years ago

  • Target version set to 8 LTS
Actions #2

Updated by Oliver Hader over 8 years ago

  • Status changed from New to Needs Feedback
Which setting takes precedence? I'd say
  • options.workspaces.previewLinkTTLHours
  • sys_workspace.preview_link_lifetime (would be a new field then)
  • default (48 hours)
Actions #3

Updated by Patrick Gaumond over 8 years ago

Which setting takes precedence? I'd say
  • options.workspaces.previewLinkTTLHours
  • sys_workspace.preview_link_lifetime (would be a new field then)
  • default (48 hours)

Just to be sure, you mean :

If nothing is done default (48h) and then if there's NO value in User TSconfig (options.workspaces.previewLinkTTLHours) but one for the new sys_workspace.preview_link_lifetime value in the WS record itself then that's the one who would be used (my original suggestion).

Right ?

Actions #4

Updated by Alexander Opitz about 8 years ago

  • Status changed from Needs Feedback to New
Actions #5

Updated by Riccardo De Contardi about 7 years ago

  • Target version changed from 8 LTS to 9.0
Actions #6

Updated by Susanne Moog over 6 years ago

  • Target version deleted (9.0)
Actions #7

Updated by Benni Mack about 4 years ago

  • Status changed from New to Needs Feedback

Current implementation is a huge problem, as it is related to the CURRENT's userTS (the user who creates the link).

Making this completely part of the workspace record makes a lot of sense to me, effectively removing the UserTS option. What do you think?

Actions #8

Updated by Patrick Gaumond about 4 years ago

Benni Mack wrote:

Current implementation is a huge problem, as it is related to the CURRENT's userTS (the user who creates the link).

Making this completely part of the workspace record makes a lot of sense to me, effectively removing the UserTS option. What do you think?

Still annoying 5 years later. Workspaces need more love and fixes.

I still thinks it should be a setting at WS level.

Actions #9

Updated by Gerrit Code Review about 4 years ago

  • Status changed from Needs Feedback to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63881

Actions #10

Updated by Gerrit Code Review about 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63881

Actions #11

Updated by Gerrit Code Review about 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63881

Actions #12

Updated by Gerrit Code Review about 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/63881

Actions #13

Updated by Benni Mack about 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #14

Updated by Benni Mack about 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF