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

outdated error clearer #634

Closed
wants to merge 7 commits into from
Closed

outdated error clearer #634

wants to merge 7 commits into from

Conversation

deathowl
Copy link

@deathowl deathowl commented Jan 8, 2014

Another database cleaning option

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling d7a4f5c on deathowl:master into 493541b on errbit:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.38%) when pulling 472a6d4 on deathowl:master into 493541b on errbit:master.

@deathowl deathowl closed this Jan 8, 2014
@deathowl
Copy link
Author

deathowl commented Jan 8, 2014

i'll fix this and reopen

@deathowl deathowl reopened this Jan 8, 2014
@deathowl
Copy link
Author

deathowl commented Jan 8, 2014

fixed and tested.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.37%) when pulling 910e64f on deathowl:master into 493541b on errbit:master.

@deathowl
Copy link
Author

Also added advanced search form, could you please reply to this pull request?

@shingara
Copy link
Member

This pull request group 2 features now So can you split it in both pull request or only add outdated here and open a new one with Advanced search ?

About the outdated. It's good but we need allow the capability to define the time of you think it's outdated. In you code you fix it to 1 week. But in some case 1 week is too soon.

Can you add this information in input text before the button or add an option by application ?

@deathowl
Copy link
Author

Advanced search separate pull request added.
Outdated cleaner parameter is now in progress. So I'll close this request

@deathowl deathowl closed this Feb 19, 2014
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.

None yet

3 participants