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

Remove postdate and fulldate columns from cpanstats table #25

Open
preaction opened this issue Jun 15, 2018 · 0 comments
Open

Remove postdate and fulldate columns from cpanstats table #25

preaction opened this issue Jun 15, 2018 · 0 comments

Comments

@preaction
Copy link
Member

These two date/time-ish fields are duplicates of created (#24), and so do not need to exist. But before they can be removed, all code that depends on them must be updated to use created instead:

  • APIs/pages that filter or sort by date should use created
  • APIs/pages that display the date should use created with the appropriate (compatible) format

API compatibility is paramount and breakage is not allowed. HTML page compatibility is nice to have: If it's an ordeal to maintain exact compatibility on any HTML, we won't. Creating new, backwards-compatible replacement APIs and pages is preferred to spending egregious amounts of time maintaining legacy APIs and pages.

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

1 participant