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

feature request: sort by due date #66

Closed
imlonghao opened this issue Sep 26, 2022 · 9 comments
Closed

feature request: sort by due date #66

imlonghao opened this issue Sep 26, 2022 · 9 comments

Comments

@imlonghao
Copy link
Contributor

Those pages /servers, /shared, and so on, currently is sorted by the random ID.

It would be better those pages can be sorted by the due date so we can know which server is going to expired.

@cp6
Copy link
Owner

cp6 commented Oct 4, 2022

In progress: #70

@imlonghao
Copy link
Contributor Author

I found this which can sort table in front-end
https://www.kryogenix.org/code/browser/sorttable/

just FYI

@cp6
Copy link
Owner

cp6 commented Oct 9, 2022

Thanks for that, ill stick with the backend sorting for now as a front-end library may likely struggle when the dates are formatted to human readable strings

@imlonghao
Copy link
Contributor Author

imlonghao commented Oct 9, 2022

https://www.kryogenix.org/code/browser/sorttable/#customkeys

human readable strings can also be sort as long as there are sorttable_customkey attribute

@imlonghao
Copy link
Contributor Author

I tried the latest version, the server order seems still fixed to me no matter how I change the order setting.

@cp6
Copy link
Owner

cp6 commented Oct 20, 2022

Was an issue with the price relationship have got this working beaf9af

@imlonghao
Copy link
Contributor Author

The Server page work as expected after change the order setting,
but it seems you forget to clear the cache for the Domain page.

@cp6
Copy link
Owner

cp6 commented Oct 31, 2022

you forget to clear the cache for the Domain page.

Fixed here a20ce98

@imlonghao
Copy link
Contributor Author

LGTM

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

No branches or pull requests

2 participants