Re: [PROJXP] [ProjXP] #114: If a user doesn't have avatar, then a default avatar is used.

[Darryl L. Pierce]

On Mon, Apr 13, 2009 at 6:25 PM, ProjXP <trac(a)fedorahosted.org&gt; wrote:
>  #114: If a user doesn't have avatar, then a default avatar is
> used.
>  --------------------------+-------------------------------------------------
>   Reporter:  mcpierce     |       Owner:  nebj
>       Type:  enhancement  |      Status:  closed
>   Priority:  major        |   Milestone:  Version 0.2.0
>   Component:  Users        |     Version:
>  Resolution:  fixed        |    Keywords:
>  --------------------------+-------------------------------------------------
>  Changes (by nebj):
>
>   * status:  assigned => closed
>   * resolution:  => fixed 
Before committing, did you apply the feedback I provided; i.e., adding
a migration to insert the default URL property into config_properties?

---8<[quote]---
I like the idea of making the default avatar configurable. However, if
we do this, there should be a migration added as well to inject the
default value. That way an admin can use the system out of the box
without having to manually inject a record.
---8<[quote]---

Otherwise, how will a system know what the default icon is to be?

-- 
Darryl L. Pierce <mcpierce(a)gmail.com&gt;
Visit the Infobahn Offramp: <http://mcpierce.multiply.com>
"Bury me next to my wife. Nothing too fancy..." - Ulysses S. Grant

[ProjXP] #114: If a user doesn't have avatar, then a default avatar is used.

[LAN-SUN-LUK Benjamin]

Yes, look at migration #31. I hope it's correct.

--
Benjamin LAN-SUN-LUK



Le 14/04/09 01:57, « Darryl Pierce » <mcpierce(a)gmail.com&gt; a écrit :

On Mon, Apr 13, 2009 at 6:25 PM, ProjXP <trac(a)fedorahosted.org&gt; wrote:
>  #114: If a user doesn't have avatar, then a default avatar is
> used.
>  --------------------------+-------------------------------------------------
>   Reporter:  mcpierce     |       Owner:  nebj
>       Type:  enhancement  |      Status:  closed
>   Priority:  major        |   Milestone:  Version 0.2.0
>   Component:  Users        |     Version:
>  Resolution:  fixed        |    Keywords:
>  --------------------------+-------------------------------------------------
>  Changes (by nebj):
>
>   * status:  assigned => closed
>   * resolution:  => fixed 
Before committing, did you apply the feedback I provided; i.e., adding
a migration to insert the default URL property into config_properties?

---8<[quote]---
I like the idea of making the default avatar configurable. However, if
we do this, there should be a migration added as well to inject the
default value. That way an admin can use the system out of the box
without having to manually inject a record.
---8<[quote]---

Otherwise, how will a system know what the default icon is to be?

--
Darryl L. Pierce <mcpierce(a)gmail.com&gt;
Visit the Infobahn Offramp: <http://mcpierce.multiply.com>
"Bury me next to my wife. Nothing too fancy..." - Ulysses S. Grant
_______________________________________________
projxp-devel mailing list
projxp-devel(a)lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/projxp-devel

[ProjXP] #114: If a user doesn't have avatar, then a default avatar is used.

[Darryl L. Pierce]

On Mon, Apr 13, 2009 at 8:00 PM, LAN-SUN-LUK Benjamin
<Benjamin.LAN-SUN-LUK(a)supinfo.com&gt; wrote:
>  Yes, look at migration #31. I hope it’s correct. 
I don't have a migration 331. Did you forget to add it to the commit?

-- 
Darryl L. Pierce <mcpierce(a)gmail.com&gt;
Visit the Infobahn Offramp: <http://mcpierce.multiply.com>
"Bury me next to my wife. Nothing too fancy..." - Ulysses S. Grant

[ProjXP] #114: If a user doesn't have avatar, then a default avatar is used.

[Darryl L. Pierce]

On Mon, Apr 13, 2009 at 9:03 PM, Darryl Pierce <mcpierce(a)gmail.com&gt; wrote:
>  On Mon, Apr 13, 2009 at 8:00 PM, LAN-SUN-LUK Benjamin
>  <Benjamin.LAN-SUN-LUK(a)supinfo.com&gt; wrote:
> > Yes, look at migration #31. I hope it’s correct.
>
>  I don't have a migration 331. Did you forget to add it to the commit? 
Typo, I meant #31 there.

I reverted the commit. Please submit a patch to the list and wait for
an ACK before pushing. Thanks.

-- 
Darryl L. Pierce <mcpierce(a)gmail.com&gt;
Visit the Infobahn Offramp: <http://mcpierce.multiply.com>
"Bury me next to my wife. Nothing too fancy..." - Ulysses S. Grant

[ProjXP] #114: If a user doesn't have avatar, then a default avatar is used.

[LAN-SUN-LUK Benjamin]

Okay, I will make a new patch ;).
--
Benjamin LAN-SUN-LUK



Le 14/04/09 11:58, « Darryl Pierce » <mcpierce(a)gmail.com&gt; a écrit :

On Mon, Apr 13, 2009 at 9:03 PM, Darryl Pierce <mcpierce(a)gmail.com&gt; wrote:
>  On Mon, Apr 13, 2009 at 8:00 PM, LAN-SUN-LUK Benjamin
>  <Benjamin.LAN-SUN-LUK(a)supinfo.com&gt; wrote:
> > Yes, look at migration #31. I hope it's correct.
>
>  I don't have a migration 331. Did you forget to add it to the commit? 
Typo, I meant #31 there.

I reverted the commit. Please submit a patch to the list and wait for
an ACK before pushing. Thanks.

--
Darryl L. Pierce <mcpierce(a)gmail.com&gt;
Visit the Infobahn Offramp: <http://mcpierce.multiply.com>
"Bury me next to my wife. Nothing too fancy..." - Ulysses S. Grant
_______________________________________________
projxp-devel mailing list
projxp-devel(a)lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/projxp-devel

[ProjXP] #114: If a user doesn't have avatar, then a default avatar is used.

[Darryl L. Pierce]

On Tue, Apr 14, 2009 at 8:43 AM, LAN-SUN-LUK Benjamin
<Benjamin.LAN-SUN-LUK(a)supinfo.com&gt; wrote:
>  Okay, I will make a new patch ;). 
Excellent. :)

-- 
Darryl L. Pierce <mcpierce(a)gmail.com&gt;
Visit the Infobahn Offramp: <http://mcpierce.multiply.com>
"Bury me next to my wife. Nothing too fancy..." - Ulysses S. Grant