Ok


On Mon, Jan 6, 2014 at 1:19 PM, Yoshiki Vázquez Baeza <
notifications@github.com> wrote:

> In www/select_study_task.psp:
>
> > @@ -327,6 +329,13 @@ elif ebi_study_accession is not None:
> >
> >  # End indent
> >  # End if
> > +if vamps_id is not None:
> > +    # Indent
> > +%>
> > +        <li>
> > +            <a href="http://vamps.mbl.edu/portals/mobedac/mobedac_cv.php?project=<%=vamps_id%>" target="_blank">View Study at VAMPS</a>
>
> Depending on what we are storing this column as, if we are going to be
> storing this string in a column named VAMPS_ID it should definitely be
> whatever VAMPS uses as an identifier, if it's MBE_#### or if it is ####just for the sake of consistency. Otherwise the name of the column would
> not really make sense right?
>
> I don't like the idea of having to format the ID in some extra piece of
> code an ID should be useful as it is.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/qiime/qiime_web_app/pull/564/files#r8676104>
> .
>