Commit Graph

24 Commits

Author SHA1 Message Date
Connor Shea 32e31d626f
Resolve feedback. 2016-08-15 11:25:28 -06:00
Douwe Maan 33edde50ab Don’t show “Resolve discussion” for non-author/members 2016-08-12 15:10:05 -05:00
Phil Hughes c51cbc1f2f Updated issues with jump tp button
Fixed styling bugs
2016-08-01 10:06:31 +01:00
Phil Hughes d147894b84 Hides jump to next button if only one discussion visible 2016-07-29 15:22:26 +01:00
Phil Hughes 9878356dbc Added JS spec tests for CommentsStore
Updated other tests
2016-07-29 15:03:58 +01:00
Phil Hughes efb74875cf Moved most of the data handling into discussion & notes models
Reduced some duplicated code with compiling components
Fixed bug with resolve button tooltip not updating after resolving discussion
2016-07-29 11:19:56 +01:00
Douwe Maan 27471d01c2 Don’t depend on unique class. 2016-07-28 20:09:50 -06:00
Douwe Maan 1c2eefef3f Backend tweaks 2016-07-28 20:09:36 -06:00
Douwe Maan 6d9715d877 Show existing discussion when adding new comment on line with a hidden resolved discussion 2016-07-28 19:28:56 -06:00
Phil Hughes 8539396216 Fixed jump tp button being hidden 2016-07-27 19:07:12 +01:00
Phil Hughes 9398ee7f2f Hides jump to next button on commit discussion 2016-07-27 18:36:26 +01:00
Phil Hughes 2247d8a4fd Updates the text above discussions when resolving notes & discussions 2016-07-27 18:34:04 +01:00
Phil Hughes 86446846e9 Added tests for resolving comments feature 2016-07-27 16:57:48 +01:00
Phil Hughes 5a0cd0e947 Added jump tp next discussion button 2016-07-27 09:27:00 +01:00
Phil Hughes e786c2b21c Comment & resolve button text is done through VueJS 2016-07-26 17:45:48 +01:00
Phil Hughes a678fef836 Added resolved by users name into tooltip 2016-07-26 15:47:19 +01:00
Phil Hughes 6537a4a8fe Correctly resolves/unresolves discussions 2016-07-26 13:44:51 +01:00
Douwe Maan aa26767897 Fix reply button on commit discussions 2016-07-25 22:49:53 -06:00
Douwe Maan b3180d3486 Add resolved and last updated status to collapsed discussions 2016-07-25 22:49:21 -06:00
Douwe Maan 3b06987c34 Hide resolve button when not allowed 2016-07-25 22:48:15 -06:00
Douwe Maan 2ac0099923 Fix discussion partials 2016-07-24 13:17:16 -06:00
Phil Hughes 44cb5803a1 Seperated resolve all button from reply button 2016-07-24 13:12:59 -06:00
Phil Hughes c1fe066b4b Added resolve button to discussions
Top count displays how many resolved discussions
2016-07-24 13:12:58 -06:00
Douwe Maan 79214be727 Add Discussion model to represent MR/diff discussion 2016-07-20 16:18:18 -06:00