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

[Issue 91]: Optionally use netid as user unique identifier #93

Merged
merged 3 commits into from
Jul 18, 2024

Conversation

wwelling
Copy link

@wwelling wwelling commented Jul 17, 2024

Migration script to update weaver_users with duplicate netid after migrating uin to netid:

WITH duplicate_netids AS (
    SELECT netid, COUNT(*) AS count FROM weaver_users GROUP BY netid HAVING COUNT(*) > 1
)
UPDATE weaver_users SET netid = NULL FROM duplicate_netids WHERE weaver_users.netid = duplicate_netids.netid AND weaver_users.email NOT LIKE '%tamu.edu';

Migration script to add unique constraint on weaver_users netid:

ALTER TABLE weaver_users ADD CONSTRAINT weaver_users_netid UNIQUE (netid);

@wwelling wwelling merged commit f7f9356 into sprint2-staging Jul 18, 2024
0 of 2 checks passed
@wwelling wwelling removed the request for review from kaladay July 19, 2024 12:34
@wwelling wwelling changed the title Optionally use netid as user unique identifier [Issue 91]: Optionally use netid as user unique identifier Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants