Issue #773: List of closed issues should show date closed - pagure

pagure

#773 List of closed issues should show date closed

Closed: Fixed None Opened 8 years ago by ryanlerch.

When showing the list of closed issues, we should also show the closure date. Since the issue is closed, who it is assigned to is really not that pertinant, so we could probably remove that column to make room for the closure date.

ryanlerch commented 8 years ago

This one is going to need new fields in the database. we currentlydont record the date of closure, or who closed an issue

pingou commented 8 years ago

Note that this is quite close to https://pagure.io/pagure/issue/774 :)

rahulbajaj commented 8 years ago

I would like to work on this bug :D

pingou commented 8 years ago

@rahulbajaj you're welcome to if you want :)

rahulbajaj commented 8 years ago

@pingou, i have a few questions-
1) Changes have to be done in both issues.html and issue,html ?
2)Can you guide me-" how to add a data field in the issue.html"
3) I have writtten a little snippet - please guide me whether it is correct or not - http://fpaste.org/348009/57078145/

Edited 8 years ago by rahulbajaj

pingou commented 8 years ago

1) Changes have to be done in both issues.html and issue.html ?

The title mentions the list of closed issues, so is issue.html listing issues?

2) You need to add a new field in the database, so that implies a change in model.py as well as an alembic migration file to update current database. If you do not know alembic, http://alembic.readthedocs.org/en/latest/tutorial.html is a good resource to check.

3) The snippet looks good indeed, once issues have an closed_at attribute :)