Skip to content
This repository has been archived by the owner on Feb 10, 2019. It is now read-only.

Flag Outcome Times #32

Merged
merged 2 commits into from
Nov 22, 2015
Merged

Flag Outcome Times #32

merged 2 commits into from
Nov 22, 2015

Conversation

mezmi
Copy link
Contributor

@mezmi mezmi commented Nov 21, 2015

Added a function to display the time a of a flags outcome, example.

spam – shu8 38 mins ago helpful – 34 mins ago

enkimatt added 2 commits November 21, 2015 15:31
When viewing your flag history the time a flag was marked as helpful, disputed or declined is not displayed by default, the included function appends the time to the flag outcome element.
Show the flag outcome time when viewing your flag history.

Example

spam – shu8 38 mins ago   helpful – 34 mins ago
@shu8 shu8 mentioned this pull request Nov 22, 2015
@shu8 shu8 merged commit ac1faa6 into shu8:develop Nov 22, 2015
@shu8
Copy link
Owner

shu8 commented Nov 22, 2015

@enkimatt thanks! added to dev version.

@shu8
Copy link
Owner

shu8 commented Nov 22, 2015

@enkimatt BTW did you get that from a feature request on Meta? If so, can you give me the link please so I can link to it in the README. If you didn't, no problem, I'll just add it without a link :)

@mezmi
Copy link
Contributor Author

mezmi commented Nov 22, 2015

@shu8 - No, it wasn't a feature request (that I know of), just something that has been bugging me lately. :)

@shu8
Copy link
Owner

shu8 commented Nov 22, 2015

@enkimatt ok, adding to the README now :)

@mezmi
Copy link
Contributor Author

mezmi commented Nov 22, 2015

Very cool! :) I have a few other features I would like to implement, hopefully you don't mind..

@shu8
Copy link
Owner

shu8 commented Nov 22, 2015

@enkimatt no problem, but please make separate issues for them first! (and if possible, links to Meta :)

@mezmi
Copy link
Contributor Author

mezmi commented Nov 22, 2015

Oh, I will! :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants