[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [sup-devel] [PATCH] Message#edit_labels [was: Ruby question: before-add-message.rb and adding multiple labels at once]

To: sup-devel <sup-devel@rubyforge.org>
Subject: Re: [sup-devel] [PATCH] Message#edit_labels [was: Ruby question: before-add-message.rb and adding multiple labels at once]
From: William Morgan <wmorgan-sup@masanjin.net>
Date: Sat, 23 Jan 2010 08:11:33 -0500
Authentication-results: mx.google.com; spf=pass (google.com: domain of sup-devel-bounces@rubyforge.org designates 205.234.109.19 as permitted sender) smtp.mail=sup-devel-bounces@rubyforge.org
Delivered-to: eg@gaute.vetsj.com
Delivered-to: sup-devel@rubyforge.org
In-reply-to: <1264117436-sup-4429@tilus.net>
List-archive: <http://rubyforge.org/pipermail/sup-devel>
List-help: <mailto:sup-devel-request@rubyforge.org?subject=help>
List-id: Sup developer discussion <sup-devel.rubyforge.org>
List-post: <mailto:sup-devel@rubyforge.org>
List-subscribe: <http://rubyforge.org/mailman/listinfo/sup-devel>, <mailto:sup-devel-request@rubyforge.org?subject=subscribe>
List-unsubscribe: <http://rubyforge.org/mailman/options/sup-devel>, <mailto:sup-devel-request@rubyforge.org?subject=unsubscribe>
References: <1263574849-sup-3477@sam.mediasupervision.de> <1263680819-sup-415@tilus.net> <1264117436-sup-4429@tilus.net>
Reply-to: Sup developer discussion <sup-devel@rubyforge.org>
Sender: sup-devel-bounces@rubyforge.org
User-agent: Sup/git

Reformatted excerpts from Tero Tilus's message of 2010-01-21:
> Did this get merged?  I got no feedback.

Sorry about that. This is very similar to the code in
ThreadViewMode#multi_edit_labels, so I would prefer a solution that
shared a codepath. It's also important to call LabelManager.<< for each
label, to make sure they appear in label-list-mode.
-- 
William <wmorgan-sup@masanjin.net>
_______________________________________________
Sup-devel mailing list
Sup-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-devel

Follow-Ups:
- Re: [sup-devel] [PATCH] Message#edit_labels [was: Ruby question: before-add-message.rb and adding multiple labels at once]
  - From: Tero Tilus <tero@tilus.net>

References:
- [sup-devel] [PATCH] Message#edit_labels [was: [sup-talk] Ruby question: before-add-message.rb and adding multiple labels at once]
  - From: Tero Tilus <tero@tilus.net>
- Re: [sup-devel] [PATCH] Message#edit_labels [was: Ruby question: before-add-message.rb and adding multiple labels at once]
  - From: Tero Tilus <tero@tilus.net>

Prev by Date: Re: [sup-devel] [PATCHv3] Saved Search Support
Next by Date: [sup-devel] [PATCH] enable ruby-prof with SUP_PROFILE environment variable
Previous by thread: Re: [sup-devel] [PATCH] Message#edit_labels [was: Ruby question: before-add-message.rb and adding multiple labels at once]
Next by thread: Re: [sup-devel] [PATCH] Message#edit_labels [was: Ruby question: before-add-message.rb and adding multiple labels at once]
Index(es):
- Date
- Thread