Skip to content

Conversation

btangmu
Copy link
Member

@btangmu btangmu commented Oct 17, 2025

-New column firstdate in db cldr_users table, add and populate if not present when ST starts up

-Already existing users get approximate dates based on the first versions in which they voted, according to vote tables from version 25 and later; users who never voted get null meaning unknown

-When new user accounts are added, firstdate will get the creation date

-Display as (account created ...) in Account Settings, with indication if approximate or unknown

-Rename Java User.last_connect to User.lastlogin for consistency with lastlogin in db cldr_users table

-New convenience function DBUtils.addColumnIfMissing; slightly refactor STFactory to use it

-Change some items in UserRegistry from public to private

-Minor edits for consistency, such as uppercase for SQL keywords

-Fix recently added bug in cldrAddUser.mjs and cldrAddUser.mjs: set orgName if not admin

-Comments

CLDR-16855

  • This PR completes the ticket.

ALLOW_MANY_COMMITS=true

-New column firstdate in db cldr_users table, add and populate if not present when ST starts up

-Already existing users get approximate dates based on the first versions in which they voted, according to vote tables from version 25 and later; users who never voted get null meaning unknown

-When new user accounts are added, firstdate will get the creation date

-Display as (account created ...) in Account Settings, with indication if approximate or unknown

-Rename Java User.last_connect to User.lastlogin for consistency with lastlogin in db cldr_users table

-New convenience function DBUtils.addColumnIfMissing; slightly refactor STFactory to use it

-Change some items in UserRegistry from public to private

-Minor edits for consistency, such as uppercase for SQL keywords

-Fix recently added bug in cldrAddUser.mjs and cldrAddUser.mjs: set orgName if not admin

-Comments
@btangmu btangmu requested a review from srl295 October 20, 2025 14:08
@btangmu btangmu marked this pull request as ready for review October 20, 2025 14:08
Copy link
Member

@srl295 srl295 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looksgood. Would be good to test a blank-slate startup (with no DB)

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.

2 participants