Opened 14 years ago
Last modified 14 years ago
#120 closed defect (FIXED)
User bio should be Markdown-enabled
| Reported by: | joar | Owned by: | Aaron Williamson |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.0.4 |
| Component: | programming | Keywords: | |
| Cc: | Parent Tickets: |
Description
The user bio should be Markdown enabled with the
``mediagoblin.util.cleaned_markdown_conversion()`` function.
Proposal:
- Add a field to User (``mediagoblin/db/models.py``) which is
named ``bio_html`` (see the description\_html field in
``MediaEntry`` [same file] for refernce).
- When the user updates the bio field
(``mediagoblin/edit/views.py:edit_profile()``), run the
``cleaned_markdown_conversion()`` and put the output in the
``bio_html`` field.
- Edit ``mediagoblin/templates/mediagoblin/utils/profile.html`` to
output ``user.bio_html`` instead of ``user.bio`` and to NOT
autoescape the output of ``{{ user.bio_html }}`` (see
@mediagoblin/templates/mediagoblin/user\_pages/media.html for
reference).
Change History (6)
comment:1 by , 14 years ago
| Milestone: | → 0.0.4 |
|---|
comment:2 by , 14 years ago
| Status: | Resolved → Feedback |
|---|
Moving state to "feedback", which I will try to give tonight.
comment:3 by , 14 years ago
So this looks good. Two things: - Would be best to just use util.cleaned\_markdown\_conversion() - We could use a migration, but my bad... I need to merge the migrations stuff. Working on that now.
comment:4 by , 14 years ago
| Status: | Feedback → Closed |
|---|
I did this, and added this as the first migration in our glorious new migrations future. Horray! Thanks Aaron! I'm excited about this change, and it was a good excuse to do a simple first migration.
comment:5 by , 14 years ago
The original url for this bug was http://bugs.foocorp.net/issues/410 .
Relations:
#337: related
Note:
See TracTickets
for help on using tickets.
