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

Add further expiration options #65

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

manvari
Copy link
Contributor

@manvari manvari commented Apr 8, 2014

Resolves #58

@sametmax
Copy link
Contributor

sametmax commented Apr 8, 2014

Ok, I understand the value of the commit, but it needs some work. First, it's way too many options for a standard installation. We want the first experience to be simple. Secondly, if somebody has another taste than you, he will have to do what you did : edit the code.

Can you add a loop in the template, looping on the DURATIONS settings ? Human readable text can be extracted by splitting the dictionary key on the underscore.

@sametmax
Copy link
Contributor

sametmax commented Apr 8, 2014

You'll probably need to cast it to a tuple list with items(), sort it, json it then include it in the < head >

@sametmax
Copy link
Contributor

Any news on this ? It just need some configurability to be accepted.

@sametmax
Copy link
Contributor

If you don't feel like doing it, put it on another branch, make another PR, and I'll do it.

@sametmax
Copy link
Contributor

securitygeneration commented on 8 Apr

It's a month off or so. The calculation should be:

3600_24_365

@@ -18,8 +18,22 @@ class Paste(object):
DIR_CACHE = set()

DURATIONS = {
u'10_mins': 10 * 60,
Copy link

Choose a reason for hiding this comment

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

datetime.timedelta(minutes=10).seconds

@Mhowser
Copy link

Mhowser commented Mar 23, 2019

Any progress on this?

@sametmax
Copy link
Contributor

sametmax commented Apr 16, 2019 via email

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.

Add more expiration options
4 participants