Re: [whimsy] branch master updated: We'll be needing the email data later

[sebb AT ASF <se...@apache.org>]

Seems to have helped speed up loading incubator.json.

On 27 December 2017 at 21:24,  <se...@apache.org> wrote:
> This is an automated email from the ASF dual-hosted git repository.
>
> sebb pushed a commit to branch master
> in repository https://gitbox.apache.org/repos/asf/whimsy.git
>
>
> The following commit(s) were added to refs/heads/master by this push:
>      new 06121b5  We'll be needing the email data later
> 06121b5 is described below
>
> commit 06121b555ce9f17df833264ede8022c53a0edd9b
> Author: Sebb <se...@apache.org>
> AuthorDate: Wed Dec 27 21:24:08 2017 +0000
>
>     We'll be needing the email data later
> ---
>  www/roster/models/committee.rb | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/www/roster/models/committee.rb b/www/roster/models/committee.rb
> index 56d8801..a3e1f3d 100644
> --- a/www/roster/models/committee.rb
> +++ b/www/roster/models/committee.rb
> @@ -8,7 +8,8 @@ class Committee
>      return if members.empty? and committers.empty?
>
>      ASF::Committee.load_committee_info
> -    people = ASF::Person.preload('cn', (members + committers).uniq)
> +    # We'll be needing the mail data later
> +    people = ASF::Person.preload(['cn', 'mail', 'asf-altEmail'], (members + committers).uniq)
>
>      lists = ASF::Mail.lists(true).select do |list, mode|
>        list =~ /^#{pmc.mail_list}\b/
>
> --
> To stop receiving notification emails like this one, please contact
> ['"commits@whimsical.apache.org" <co...@whimsical.apache.org>'].