Edlib icon indicating copy to clipboard operation
Edlib copied to clipboard

fix content lock expiry seconds/minutes confusion

Open emmachughes opened this issue 3 years ago • 2 comments

Despite being commented with // seconds, the expiry time for content locks was being treated as minutes in the scheduled job that removes the locks.

To avoid future confusion, the constant is renamed to make the unit of time explicit. Since no one complained about the 90 minute expiry time, we assume this was the original intention and leave it as is.

emmachughes avatar Aug 18 '22 13:08 emmachughes

See issue https://github.com/cerpus/Edlib/issues/466

chrieinv avatar Aug 19 '22 05:08 chrieinv

Should we remove this PR?

maxmoeschinger avatar Feb 26 '25 12:02 maxmoeschinger

Obsoleted by 729032e0efd7a188a96e22d27dce843361437562

emmachughes avatar Apr 30 '25 09:04 emmachughes