Skip to content

Correct db casing #1995

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.

Sign up for GitHub

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

Jump to bottom
Merged
dannyprikaz merged 22 commits into from
Aug 8, 2025
Merged

Correct db casing #1995

dannyprikaz merged 22 commits into from
Aug 8, 2025

Conversation

@dannyprikaz
Copy link
Member

@dannyprikaz dannyprikaz commented Aug 8, 2025 •
edited
Loading

Fixes #1456

What changes did you make and why did you make them ?

  • Add script to remove duplicate users based on casing of emails
  • The database will only accept lowercase emails to ensure that no duplicate users are created.

freaky4wrld and others added 21 commits March 13, 2024 22:14
@freaky4wrld
feat: added scripts to backend folder with json data
9c349e8
@freaky4wrld
feat: added test-data.json file
fd33817
@pluto-bell
.toLowerString() added
2c09b8d
@pluto-bell
validate email addition
2a4b402
@pluto-bell
removed validate email, read note in pr
66f9339
@JackHaeg
Merge branch 'development' into add-tolowercase-db-query
0e6aae6
@trillium
Merge branch 'pr/1612' into correct-db-casing
14e5767
@trillium
Merge branch 'pr/1811' into correct-db-casing
8fd5530
@dannyprikaz
Enforce that emails can only be stored in lowercase
2975599
@dannyprikaz
Merge branch 'development' of https://github.com/hackforla/vrms into …
41fe687 
…enforce-lowercase-db
@dannyprikaz
Implement script to identify problem users
6631aca
@dannyprikaz
Add method for merging duplicate users
5017c2e
@dannyprikaz
Merge branch 'development' of https://github.com/hackforla/vrms into …
040065a 
…enforce-lowercase-db
@dannyprikaz
Add test to ensure user model sets emails to lowercase
a6ff776
@dannyprikaz
Merge pull request #1979 from dannyprikaz/enforce-lowercase-db
2b9223c 
Enforce that emails can only be stored in lowercase
@dannyprikaz
Add functionality to update checkins with new user_id
06ed72f
@dannyprikaz
Merge branch 'development' of https://github.com/hackforla/vrms into …
1520406 
…db-script
@dannyprikaz
Merge branch 'correct-db-casing' of https://github.com/hackforla/vrms …
0328e87 
…into db-script
@dannyprikaz
Use sets to keep array fields from getting duplicate entries
53a294f
@dannyprikaz
Remove mongo url from code, and clean up notes
c528c69
@trillium
Merge pull request #1991 from dannyprikaz/db-script
0d36b7a 
Db script
trillium
trillium approved these changes Aug 8, 2025
View reviewed changes
Copy link
Member

@trillium trillium left a comment

Choose a reason for hiding this comment

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

Reviewed live with Danny

👍

@dannyprikaz dannyprikaz merged commit 05ff12c into development Aug 8, 2025
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@trillium trillium trillium approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Epic: DB accepts capital and lower case email credentials for email login

6 participants

@dannyprikaz @trillium @freaky4wrld @pluto-bell @JackHaeg