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

Is there a way to default to keep forever? I hate having to change it every time, and frequently forget to, making links die. #282

Open
levihb opened this issue May 20, 2022 · 4 comments · May be fixed by #322

Comments

@levihb
Copy link

levihb commented May 20, 2022

I use this to share online, and I would generally prefer it to be up forever, especially when I share in forums etc. But sometimes I forget to change it from one month to keep forever.

And also it's just annoying having to change it. Is there some way to just default to forever?

@ThomasWaldmann
Copy link
Contributor

https://github.com/bepasty/bepasty-server/blob/master/src/bepasty/utils/date_funcs.py#L8

That's the place in the source - can you make a PR to make this configurable?

Back then, when I implemented that, I intentionally did not set "forever" as default to get a self-cleaning-by-default behaviour.

@levihb
Copy link
Author

levihb commented May 20, 2022

https://github.com/bepasty/bepasty-server/blob/master/src/bepasty/utils/date_funcs.py#L8

That's the place in the source - can you make a PR to make this configurable?

Back then, when I implemented that, I intentionally did not set "forever" as default to get a self-cleaning-by-default behaviour.

Ahh cheers. Sure I'll try and get around to it on Sunday. Do you have any specific way you'd like it configurable? Maybe a setting with a value of MINUTES,30 to set it to default to 30 mins, etc?

@ThomasWaldmann
Copy link
Contributor

Yes, for example. See other places that deal with configured values.

@ThomasWaldmann
Copy link
Contributor

@levihb do you want to make a PR for this?

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 a pull request may close this issue.

2 participants