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 user alias from database #13547

Closed
diosmosis opened this issue Oct 5, 2018 · 6 comments · Fixed by #15714
Closed

Remove user alias from database #13547

diosmosis opened this issue Oct 5, 2018 · 6 comments · Fixed by #15714
Assignees
Milestone

Comments

@diosmosis
Copy link
Member

Now that user alias has been removed from the UI, it can be removed from the DB as well. This should be done for Matomo 4.

@diosmosis diosmosis added this to the 4.0.0 milestone Oct 5, 2018
@tsteur
Copy link
Member

tsteur commented Oct 5, 2018

Actually, it could be removed in 3.7 or so as well as it isn't a DB update on a major big table.

@diosmosis
Copy link
Member Author

diosmosis commented Oct 5, 2018

Some users might be making use of it, though? At least through the API or something.

@tsteur
Copy link
Member

tsteur commented Oct 5, 2018

👍

@Legion2
Copy link

Legion2 commented Dec 12, 2018

Please don't remove user alias from database, it's the only field where you can store custom data. We are currently using it to store the users full name and the login is only a short department id.
We are also planning to use the loginldap plugin and then the login is only an unique user id form ldap which doesn't contain any information about the full name.

If the alias is removed we have big problems managing the users permissions.

@sgiehl
Copy link
Member

sgiehl commented Feb 14, 2020

@tsteur shall we remove the alias in Matomo 4? 🤔

@tsteur
Copy link
Member

tsteur commented Feb 16, 2020

Yes, I was going to wait though for the token auth branch (#6559) to be merged first to avoid merge conflicts. Be good to in the meanwhile work on some analytics features maybe.

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 a pull request may close this issue.

4 participants