summaryrefslogtreecommitdiffstats
path: root/app/controllers/comments_controller.rb
Commit message (Collapse)AuthorAgeFilesLines
* Support for watching a merge request version through commenting.Marius Mathiesen2009-12-151-1/+5
| | | | Also (finally) added support for login_as(<User>)
* Make it possible to start watching a merge request when submitting a comment ↵Marius Mathiesen2009-12-151-0/+5
| | | | | | on it. Currently only for non-inline comments, this is up next
* MergeRequest's to_param now uses the sequence number instead of idMarius Mathiesen2009-11-301-1/+1
| | | | | | | | | | A summary of what's changed: - Routing - Links, where necessary - Git branch names use seq instead of id Also made some refactorings in PushEventProcessor to make it testable without a full stack test. And set some revealing sequence numbers in the fixtures.
* Render the new comment and diffs when inline commenting on a commitMarius Mathiesen2009-11-161-8/+12
| | | | - Made partials/javascripts reusable between different inline commenting objects
* Make Comments#create render something meaningful when receiving a comment on ↵Marius Mathiesen2009-11-161-18/+36
| | | | | | a commit - A minor refactoring of this whole rendering was indeed required
* If creating a merge-request comment, render two partials into a JSON structureJohan Sørensen2009-11-101-1/+4
| | | | | So we can update both the file-diff and the .comments div with the "normal" comment list
* Making some methods protected in the CommentsController, and fixed bad ↵Johan Sørensen2009-11-101-47/+48
| | | | indentation
* Only notify the merge request owner once on new commentsJohan Sørensen2009-11-051-13/+0
| | | | | | | There's already an after_save hook in Comment for notifying people, the controller should not do this as well. Further more the tests for this was invalid since it used the same user as the creator of the merge request (he shouldn't get notified about his own comments).
* Make sure we re-render the diff correctly when commenting on single commitJohan Sørensen2009-11-041-5/+16
|
* Support for editing comments for the author 10 minutes after creating themMarius Mathiesen2009-11-041-0/+18
| | | | | - A new link is displayed when hovering over an editable comment - Clicking on this loads the edit form, and makes it submit over ajax
* Create an Event when commenting on a MergeRequestVersion as wellJohan Sørensen2009-11-041-3/+9
|
* When POSTing a comment on a MR version, the controller renders the relevant ↵Marius Mathiesen2009-11-041-1/+3
| | | | | | diff, which is displayed via JS - Needed to extract the diff browsing to a separate helper as other helpers include each other
* Merge request owners are notified of new comments on their code.Marius Mathiesen2009-11-041-1/+15
|
* Non-authenticated users cannot comment on merge request versions.Marius Mathiesen2009-11-041-1/+0
|
* Code cleanup in CommentsControllerMarius Mathiesen2009-11-041-10/+13
| | | | | Note: all ajax requests are now sent with text/javascript Accept headers. This means they are interpreted as format.js in the controllers
* Handle errors on creation of commentsMarius Mathiesen2009-11-041-2/+9
|
* Adding support for commenting on a merge request version (UI still needs ↵Marius Mathiesen2009-11-041-3/+3
| | | | some love)
* Support for commenting on Merge Request versions.Marius Mathiesen2009-11-041-0/+4
|
* Only create a single event when creating a comment with a new stateMarius Mathiesen2009-07-131-1/+1
|
* Let each project define its status tags for merge requestsMarius Mathiesen2009-07-021-1/+1
| | | | - when changing the state from a comment, create an event - display status field in merge request form
* Adding a loose state (status_tag) to merge requests, adding state change to ↵Marius Mathiesen2009-06-241-0/+2
| | | | | | | | | | | comments. - Adding a comment to a merge request now potentially sets or changes the loose state of the merge request (target) This way, target repository owners can mark that a merge request should change state when adding comments. This will eventually replace the more final reason from merge requests, replacing it with comments with an added context
* Do extra work to make sure clones are scoped within the correct projectJohan Sørensen2009-05-101-1/+1
|
* Handle displaying an event properly when a comment is posted on a MergeRequestJohan Sørensen2009-04-291-2/+10
|
* Be able to comment on merge requests as wellJohan Sørensen2009-04-291-5/+22
| | | | | This requires the comments_controller to be able to deal with the polymorphic targets on the comment
* Adding preview of commentsMarius Mathiesen2009-04-221-0/+7
|
* Rethinking the relationship between comments and its events:Marius Mathiesen2009-04-221-1/+2
| | | | | | | - The event created for a comment now belongs to the repository - The comment's id is stored in the Event's data property, ensuring the relationship can be held Added a migration that migrates existing data into this new regime. Should also be reversable.
* Updated copyrights-blurb in all application filesJohan Sørensen2009-04-221-0/+4
|
* Rename install_site_before_filters to renders_in_site_specific_context andJohan Sørensen2009-04-221-1/+1
| | | | | | | introduce a renders_in_global_context controller class method as well. This lets us control in the Site context in which specific actions or controllers as a whole are rendered in.
* Redirect to the current_site's subdomain, if it has one and it's a GET requestJohan Sørensen2009-04-221-0/+1
|
* Move commit comments into the commit view, and prettify things a bitJohan Sørensen2009-04-221-11/+7
|
* Prettify listing of commit message + committer/authorJohan Sørensen2009-04-221-0/+2
|
* Moved the entire test suite from RSpec to Test::Unit + ShouldaJohan Sørensen2009-04-221-2/+2
|
* Update to Rails 2.3RC1Johan Sørensen2009-04-221-1/+1
|
* Major refactoring of the whole Gitorious project. I've added both ↵Fabio Akita2009-01-091-1/+1
| | | | localized_date and localied_templates plugin and I've extracted every single string that could be translated. Now you have both locales/en.rb and locales/pt-BR.rb as well as views/pt-BR for localized static pages and email templates. I have not added a locale field in the User model - yet, so you have to change gitorious.yml to change it globally
* Add copyright and license headers (AGPL) to code filesTor Arne Vestbø2008-11-281-0/+18
| | | | | The authors in each copyright header was computed by running 'git log --reverse' on that particular file.
* Moved events to be primarily pivoting around the project.Johan Sørensen2008-04-211-1/+1
|
* Removed actions from the databaseDavid A. Cuadrado2008-04-161-1/+1
| | | | Signed-off-by: David A. Cuadrado <krawek@gmail.com>
* refactoring to use polymorphic associations (USER does ACTION on X)David A. Cuadrado2008-04-071-1/+1
| | | | Signed-off-by: David A. Cuadrado <krawek@gmail.com>
* Implemented user event systemDavid A. Cuadrado2008-03-201-0/+1
|
* MergeRequest wrapup of views and permissionsJohan Sørensen2008-03-021-1/+1
|
* Added MergeRequest views and relatedJohan Sørensen2008-03-021-0/+1
|
* Updated repo browser and other git lib related thingsJohan Sørensen2008-02-221-1/+1
| | | | | | - breadcrumbs - made it work with non-master branches - made it work when the only branch is a non-master
* - Clean up commit formatting a wee bitJohan Sørensen2008-02-061-2/+2
| | | | - use Gitto in (Comments|Repositories)Controller as well
* add a project_id to comments and the beginnings of a dashboard for logged in ↵Johan Sørensen2008-01-261-0/+1
| | | | users
* Fixed Atom url generationJohan Sørensen2008-01-121-1/+1
|
* atom feed for projects+tags and repository commentsJohan Sørensen2008-01-071-0/+5
|
* Added commenting of repositories and commitsJohan Sørensen2008-01-071-0/+39
generated by cgit v1.1 at 2025-05-13 23:46:08 +0000