Skip to content

Feat: Merge Conf_Admin and Admin #1107#1115

Open
lcanobbio wants to merge 9 commits intomasterfrom
feature/1107-merge-conf_admin-and-admin
Open

Feat: Merge Conf_Admin and Admin #1107#1115
lcanobbio wants to merge 9 commits intomasterfrom
feature/1107-merge-conf_admin-and-admin

Conversation

@lcanobbio
Copy link
Copy Markdown
Collaborator

No description provided.

Miguel7373 and others added 7 commits March 30, 2026 10:24
* implement a first version of cancancan

* Finish giving rights

* Fix bug that unifing skills was for admins but in a view only accessible by conf_admins

* remove pry

* Remove user specific rights

* Move some other user_classes in to the user_classes method

* Add contributions to user rights

* Write logic for people only being able to edit their own profile

* Correctly save ldap username on person

* Modify ldap_username to be correct in api and make rubocop happy

* Add tests for access control and setup users for testing

* Feat: implement cancancan considering all roles #947

* feat: remove form from people skills if you dont have the right to change them #947

* Refactor: resolve conversations #947

* Fix: has one not working #947

* Fix: has one dependent #947

* test: fix failing capybara tests #947

* test: fix fixture rights #947

* test: add tests for editor #947

* refactor(fixture): update to the right syntax #947

* refactor: fix permissions and add flash message #947

* test: fix tests because of new success message #947

* refactor: make code more readable #947

* feat: move message and add changelog #947

* fix: change translation methode #947

* fix: move translation #947

---------

Co-authored-by: Nevio Di Gennaro <digennaro@puzzle.ch>
Co-authored-by: Miguel Lehmann <lehmann@puzzle.ch>
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.

Access control: Merge Conf_Admin and Admin

3 participants