Skip to content

Infraction Search Dates#438

Merged
MarkKoz merged 6 commits into
masterfrom
hemlock-infraction-date-humanization
Sep 18, 2019
Merged

Infraction Search Dates#438
MarkKoz merged 6 commits into
masterfrom
hemlock-infraction-date-humanization

Conversation

@MrHemlock
Copy link
Copy Markdown
Member

Since the rework was launched, the current format for the Created and Expires dates on the infraction search embeds use the standard ISO format, which is a bit of a bear to read.

This PR proposes changing that format to "%Y-%m-%d %H:%M" which will appear as 2019-09-18 13:59

- Changed the date returned on infraction searches to use the `"%c"` strftime format instead of the standard ISO format.

Signed-off-by: Daniel Brown <browndj3@gmail.com>
- Changed to use the format `"%Y-%m-%d %H:%M"`, which will turn out looking like `2019-09-18 13:59`

Signed-off-by: Daniel Brown <browndj3@gmail.com>
@jchristgit jchristgit self-requested a review September 18, 2019 20:23
@jchristgit jchristgit self-assigned this Sep 18, 2019
@jchristgit jchristgit added area: cogs a: frontend Related to output and formatting labels Sep 18, 2019
Comment thread bot/cogs/moderation.py Outdated
Copy link
Copy Markdown
Contributor

@jchristgit jchristgit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

duck pond

- Changed the if statement to use `is None` for code clarity.

Signed-off-by: Daniel Brown <browndj3@gmail.com>
Comment thread bot/cogs/moderation.py Outdated
- Corrected an error that would have made the code bug out.  Moved a closing parentheses to the correct spot.

Signed-off-by: Daniel Brown <browndj3@gmail.com>
@MarkKoz MarkKoz merged commit ad86c73 into master Sep 18, 2019
@MarkKoz MarkKoz deleted the hemlock-infraction-date-humanization branch September 18, 2019 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: frontend Related to output and formatting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants