https://www.redmine.org/https://www.redmine.org/favicon.ico?16793021292010-09-14T15:49:21ZRedmineRedmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=202892010-09-14T15:49:21ZFelix Schäfer
<ul></ul><p>I can confirm that, I'll try to have a look tonight.</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=202962010-09-14T21:30:31ZFelix Schäfer
<ul></ul><p>I feel a little weird giving a link to github here, but the fix for that is here: <a class="external" href="http://github.com/thegcat/redmine/commit/6c2d9d79b6d60e683e3e65d45efdb3dcf54da379">http://github.com/thegcat/redmine/commit/6c2d9d79b6d60e683e3e65d45efdb3dcf54da379</a></p>
<p>Eric: tag, you're it.</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=202972010-09-14T21:41:32ZFelix Schäfer
<ul><li><strong>Assignee</strong> set to <i>Eric Davis</i></li></ul> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=203492010-09-16T01:34:28ZMischa The Evil
<ul></ul><p>Felix Schäfer wrote:</p>
<blockquote>
<p>[...]the fix for that is here: <a class="external" href="http://github.com/thegcat/redmine/commit/6c2d9d79b6d60e683e3e65d45efdb3dcf54da379">http://github.com/thegcat/redmine/commit/6c2d9d79b6d60e683e3e65d45efdb3dcf54da379</a></p>
</blockquote>
<p>I can confirm this issue too. It is introduced in <a class="changeset" title="Refactor: extract ProjectsController#activity to a new Activities controller." href="https://www.redmine.org/projects/redmine/repository/svn/revisions/4047">r4047</a>. The proposed fix works flawlessly.</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=203962010-09-16T21:23:13ZJean-Baptiste Barth
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Resolved</i></li><li><strong>Assignee</strong> changed from <i>Eric Davis</i> to <i>Jean-Baptiste Barth</i></li><li><strong>Target version</strong> set to <i>1.0.2</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li><li><strong>Resolution</strong> set to <i>Fixed</i></li></ul><p>Applied in <a class="changeset" title="Fix links to activity pages broken with r4047. #6392 Contributed by Felix Schafer" href="https://www.redmine.org/projects/redmine/repository/svn/revisions/4092">r4092</a>, thanks. I propose you assign this kind of really simple fixes to me, so that Eric can concentrate on great refactorings (I imagine his time is precious and we shouldn't waste it..). May I also suggest some of you ask commit bit to JPL ? It would be great for these kind of things at least.</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=204132010-09-17T07:07:42ZFelix Schäfer
<ul></ul><p>Jean-Baptiste Barth wrote:</p>
<blockquote>
<p>I propose you assign this kind of really simple fixes to me, so that Eric can concentrate on great refactorings (I imagine his time is precious and we shouldn't waste it..). May I also suggest some of you ask commit bit to JPL ? It would be great for these kind of things at least.</p>
</blockquote>
<p>Well, I had discussed this with Eric on IRC and he was OK with it if I assigned this kind of small things his refactorings introduced to him (it was his fault after all ;-) ). If you say you're ok with applying them too, I could either just add both of you as watchers, whomever comes first can apply it, or even assign them directly to you.</p>
<p>Regarding commit rights: I wouldn't really mind, but I think pushing the changes to github so you can pull/cherry-pick them is just as good for now (at least if you use git too…). Do you use git or svn?</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=204142010-09-17T08:12:46ZJean-Baptiste Barth
<ul></ul><p>Felix Schäfer wrote:</p>
<blockquote>
<p>Well, I had discussed this with Eric on IRC and he was OK with it if I assigned this kind of small things his refactorings introduced to him (it was his fault after all ;-) ). If you say you're ok with applying them too, I could either just add both of you as watchers, whomever comes first can apply it, or even assign them directly to you.</p>
</blockquote>
<p>OK. Sorry I didn't know you already discussed it. As you want, I receive and read all tickets anyway. An explicit sentence in the note is perfect for me.</p>
<blockquote>
<p>Regarding commit rights: I wouldn't really mind, but I think pushing the changes to github so you can pull/cherry-pick them is just as good for now (at least if you use git too…). Do you use git or svn?</p>
</blockquote>
<p>Git (anybody uses SVN nowadays ?!?). I may be wrong, but I think it takes time to review, apply, (d)commit the patch, and finally update the ticket accordingly. If we have 5, 8, 10 committers, the workload is distributed, even if 1 or 2 leaders should still decide what to merge in stable branch. My 2 euro cents...</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=204162010-09-17T11:34:50ZFelix Schäfer
<ul></ul><p>Jean-Baptiste Barth wrote:</p>
<blockquote>
<p>Felix Schäfer wrote:</p>
<blockquote>
<p>Well, I had discussed this with Eric on IRC and he was OK with it if I assigned this kind of small things his refactorings introduced to him (it was his fault after all ;-) ). If you say you're ok with applying them too, I could either just add both of you as watchers, whomever comes first can apply it, or even assign them directly to you.</p>
</blockquote>
<p>OK. Sorry I didn't know you already discussed it. As you want, I receive and read all tickets anyway. An explicit sentence in the note is perfect for me.</p>
</blockquote>
<p>Noted, then I'll have you as a target for small changes now ;-)</p>
<blockquote><blockquote>
<p>Regarding commit rights: I wouldn't really mind, but I think pushing the changes to github so you can pull/cherry-pick them is just as good for now (at least if you use git too…). Do you use git or svn?</p>
</blockquote>
<p>Git (anybody uses SVN nowadays ?!?). I may be wrong, but I think it takes time to review, apply, (d)commit the patch, and finally update the ticket accordingly. If we have 5, 8, 10 committers, the workload is distributed, even if 1 or 2 leaders should still decide what to merge in stable branch. My 2 euro cents…</p>
</blockquote>
<p>I'm fine with whatever suits Eric and you. I think I shouldn't be the one asking for the commit rights though, but rather that one or both of you tell JPLang who you think should get them. Anyway, let's switch to the forums if you want to discuss this more, I think we have polluted this ticket enough already :-)</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=204182010-09-17T11:53:18ZFelix Schäfer
<ul></ul><p>JBB: and while we're at it: in <a class="changeset" title="Fix links to activity pages broken with r4047. #6392 Contributed by Felix Schafer" href="https://www.redmine.org/projects/redmine/repository/svn/revisions/4092">r4092</a>, you committed the file <code>special_page_index.rhtml</code> to the root of the repository (well, to trunk/ in svn…), it doesn't need to be there ;-)</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=204192010-09-17T12:10:38ZJean-Baptiste Barth
<ul></ul><p>Damned. I'll fix it tonight, thanks..</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=204422010-09-17T23:31:40ZEric Davis
<ul></ul><p>Jean-Baptiste Barth wrote:</p>
<blockquote>
<p>Git (anybody uses SVN nowadays ?!?). I may be wrong, but I think it takes time to review, apply, (d)commit the patch, and finally update the ticket accordingly. If we have 5, 8, 10 committers, the workload is distributed, even if 1 or 2 leaders should still decide what to merge in stable branch. My 2 euro cents...</p>
</blockquote>
<p>I think we should discuss moving at least the patch application process to git(hub). Felix just posted his git repo there so I can now cherry-pick and commit easily. I don't care who applies the (reviewed) patch as long as it makes it back into svn and the issue is updated correctly (Target Version, Resolved, etc).</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=205132010-09-20T03:51:34ZEric Davis
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li></ul><p>Merged into 1.0-stable for release in 1.0.2</p> Redmine - Defect #6392: Updated/Created links to activity brokenhttps://www.redmine.org/issues/6392?journal_id=331622011-10-25T10:48:57ZEdmund Stewart
<ul></ul><p>SPAM</p>