Code review comment for lp:~dholbach/harvest/504299

Revision history for this message
Daniel Holbach (dholbach) wrote :

> Oh, looking at the admin interface I see that the explanation field has help
> text saying it's a link. So, I was just using it wrong :)
> I had written an explanation itself in that field earlier, so the link didn't
> work for me.
> In that case, Django does have a URLField available we can use:
> http://docs.djangoproject.com/en/dev/ref/models/fields/#urlfield

Will do.

> Another option is to store the explanation itself on the object, just like we
> do with the description. That link on the help icon would lead to something
> like "/opportunities/help/opplist/bitesize", and that page (doesn't need to be
> anything fancy — yet) would show the opportunity list's explanation. An
> advantage there is, at some point, it's trivial to have the explanation as a
> little popup instead of a whole new page.

The problem with that is that we have problems getting that data in. Shall Harvest Admins put it into the db through the admin interface? Should it be part of the harvest-data .csv files? We have some docs already for things like merges (https://wiki.ubuntu.com/UbuntuDevelopment/Merging) - it'd be sweet to just link to that info. :-)

« Back to merge proposal