Apache Allura™

Forge software for hosting software projects

Brought to you by: alexluberg, brondsem, ccruz, deshani, and 11 others

#7593 Allow site admins to add user audit entries

Milestone: v1.2.0

Status: closed

Owner: Igor Bondarenko

Labels: 42cc (432)

Component: General

Reviewer: Dave Brondsema

Updated: 2015-08-20

Created: 2014-07-25

Creator: Dave Brondsema

Private: No

On the user auditlog pages (e.g. /nf/admin/users?username= as set up by [#7524]), add the ability for the admin users to enter comments as new audit log entries. Automatically include the username of the user entering the comment. Maybe something like "Comment by {c.user.username}: {comment}"

Related

Tickets: ~~#7524~~

Discussion

Dave Brondsema - 2014-08-06

labels: --> 42cc

status: open --> in-progress

assigned_to: Igor Bondarenko
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2014-08-08

status: in-progress --> code-review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2014-08-08

Closed #627. je/42cc_7593

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2014-08-15

status: code-review --> in-progress

QA: Dave Brondsema
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2014-08-15

I would make the by param on comment_user required (or move the method out of a model). I think we should be trying to minimize and remove c.user (and app and project) usage in our models.

Line breaks in the textarea don't end up rendering as such. Maybe the audit log display should convert linebreaks to <br>? Or maybe even go so far as rendering through our markdown pipeline? (I don't think that is too important but would be nice if it has no adverse affects)

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2014-08-19

status: in-progress --> code-review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2014-08-19

Closed #636. Force-pushed je/42cc_7593

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2014-08-19

status: code-review --> closed

Milestone: forge-backlog --> forge-aug-22
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2014-08-19

I changed markdown rendering to nl2br, see commit e0ab435c03caea0b9498b7f0b961e56509b70733 for details.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2015-01-05

Milestone: unreleased --> asf_release_1.2.0
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.