Skip to content

[FIX] Create findOneByRolesAndType function#24308

Merged
matheusbsilva137 merged 3 commits intofix/24148-respect-default-role-settingfrom
fix/respect-default-user-roles
Feb 10, 2022
Merged

[FIX] Create findOneByRolesAndType function#24308
matheusbsilva137 merged 3 commits intofix/24148-respect-default-role-settingfrom
fix/respect-default-user-roles

Conversation

@matheusbsilva137
Copy link
Copy Markdown
Contributor

@matheusbsilva137 matheusbsilva137 commented Jan 27, 2022

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@matheusbsilva137 matheusbsilva137 changed the title Create findOneByRolesAndType function [FIX] Create findOneByRolesAndType function Jan 27, 2022
@matheusbsilva137 matheusbsilva137 merged commit feb99a2 into fix/24148-respect-default-role-setting Feb 10, 2022
@matheusbsilva137 matheusbsilva137 deleted the fix/respect-default-user-roles branch February 10, 2022 20:17
matheusbsilva137 added a commit that referenced this pull request Feb 18, 2022
…4173)

* Fix role handling if setup wizard pending and no admin found

Signed-off-by: Debdut Chakraborty <[email protected]>

* [FIX] Create `findOneByRolesAndType` function (#24308)

* Create findOneByRolesAndType function and stop adding user role by default

* Remove logs

* Remove default user role in userHandler.js

* Update app/authentication/server/startup/index.js

Co-authored-by: Debdut Chakraborty <[email protected]>

* fix lint issues

* Check if the server has an admin user before assigning the admin role on server startup

Co-authored-by: Matheus Barbosa Silva <[email protected]>
Co-authored-by: matheusbsilva137 <[email protected]>
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.

1 participant