refactor
working on user details