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

Fix: unique tables #1553

Merged
merged 1 commit into from
Jan 2, 2021
Merged

Fix: unique tables #1553

merged 1 commit into from
Jan 2, 2021

Conversation

JamesPSS
Copy link
Contributor

@JamesPSS JamesPSS commented Jan 2, 2021

This is fix so that username can't be the same.

  • some other tables that need to be uniqu.

You can change the username if you staff to anything whitout checks and the system don't handle that well.

It will be smart that mails is unique to but with now when you create a new site the system use the same mails 3 times so i did not add the unique mail.

@HDVinnie
Copy link
Collaborator

HDVinnie commented Jan 2, 2021

@JamesPSS I agree and thanks for the PR. The database schema will need tweaking soon with uniques and proper indexing.

@HDVinnie HDVinnie merged commit 13ba949 into HDInnovations:master Jan 2, 2021
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