Feature #170

Create a usergroup implementation

Added by Paul Kelly almost 2 years ago. Updated over 1 year ago.

Status:Closed Start:03/03/2010
Priority:Normal Due date:
Assigned to:Paul Kelly % 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.

170.patch (22.3 KB) Ohad Levy, 05/16/2010 04:08 pm

patch-squashed.diff (59.1 KB) Ohad Levy, 05/17/2010 06:52 am


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
Added by Paul Kelly over 1 year ago

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

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

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 patch-squashed added

newer version of the patch

Updated by Ohad Levy over 1 year ago

  • File deleted (patch-squashed)

Updated by Ohad Levy over 1 year ago

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

Updated by Ohad Levy over 1 year ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF