Feature #170
Create a usergroup implementation
| Status: | Closed | Start: | 03/03/2010 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assigned to: | % Done: | 100% |
||
| Category: | Web Interface | |||
| Target version: | 0.1-5 | |||
| Backlog: | Difficulity: | |||
| Votes: | 0 |
Description
Create the MVC and testing infrastructure to support the usergroup concept.
Related issues
| related to Foreman - Feature #139: per host authorization for making changes. | Closed | 12/18/2009 | ||
| precedes Foreman - Feature #73: add support for user groups | Closed | 03/04/2010 | 03/04/2010 |
Associated revisions
Revision 9c0e127b1d42b7243403dd49d739cf8e97baa60b
Fixes #170 - implement usergroups
Added migration
Added tests
Added MVC files
Added the routes as well
Added the program logic and additional views
Added RESTful actions for update and create
Added validations to ensure that usergroup.name and user.login do not collide
Added "recipients" method to users and usergroups. This returns a list of addresses.
Usergroup#recipients will follow all nested usergroups avoiding any loops and
return only unique addresses
Fixes #73 - Add usergroup support to hosts
Fixes #244 - Remove ActiveScaffold from the usergroup pages
and finally - added owners to host edit page and other whitespaces and
mods by ohad
Revision 8b62e7f33eb3110be1cb260a0d775d654e29a55a
fixes typo in migration, refs #170
History
Updated by Paul Kelly almost 2 years ago
- Status changed from New to Ready For Testing
This has been implemented in feature/73 on my github against the current edge. This includes the migration, MVC and tests. It is ready to be used by any model that wishes to use usergroups.
It uses has_may_polymorphs and integrates with ActiveScaffold for error returns.
Updated by Ohad Levy almost 2 years ago
the code looks great :)
Did you create a new feature because you didnt implement the host part?
can you switch the git submodule to .git?
please rebase to have one commit which closes the ticket ;)
thanks
Updated by Ohad Levy almost 2 years ago
- Target version set to 0.1-5
Updated by Paul Kelly almost 2 years ago
- Branch set to feature/170-usergroups
Rebased against develop.
Updated by Ohad Levy over 1 year ago
- File 170.patch added
Hi Paul,
I've started to work on some fixes to this, attached is my work in progress if you have time to continue...
Updated by Ohad Levy over 1 year ago
- File deleted (
patch-squashed)
Updated by Ohad Levy over 1 year ago
- File patch-squashed.diff added
Ohad Levy wrote:
newer version of the patch
added as diff so you can see it inline
Updated by Paul Kelly over 1 year ago
- % Done changed from 0 to 100
Applied in changeset 9c0e127b1d42b7243403dd49d739cf8e97baa60b.
Updated by Ohad Levy over 1 year ago
- Status changed from Ready For Testing to Closed
