On 2010-05-27 08:47:28 AM, Paul W. Frields wrote:
> On Thu, May 27, 2010 at 02:44:02AM -0500, Sijis Aviles wrote:
> > On Wed, May 26, 2010 at 11:40 PM, Sijis Aviles <[email protected]> 
> > wrote:
> > > I'm all for this. I think the changes are great.
> > > +1
> > >
> > > A quick thought about that architecture tab would be replacing it with an 
> > > advanced tab, which would include the stuff in get-fedora-all. We can 
> > > discuss this further.
> > >
> > > One question - what would we do about the older releases?
> > >
> > > Sijis
> > >
> > 
> > I just looked at the patch and it answers the older release question.
> > 
> > The one thing i did notice was there were a few links to
> > "http://localhost:5000/get-fedora-all";. Those should probably be
> > changed to "${lang}/get-fedora-all".
> 
> Don't we use "./get-fedora-all"?  That should land in the correct
> language for everyohne.
We usually use paths relative to /, so the correct convention would be:

${path}/${lang}/get-fedora-all

We should probably do a check of the current site to make sure we are
consistent with this.

Thanks,
Ricky

Attachment: pgpVevgz3zgYX.pgp
Description: PGP signature

-- 
websites mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/websites

Reply via email to