New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updated the discussion post to show the correct time #172

Merged

JmScherer merged 1 commit into main from fixing-discussion-display-time

May 6, 2024

Collaborator

JmScherer commented May 3, 2024 •

edited by SeriousHorncat

Loading

Checklist before requesting a review

I have performed a self-review of my code.
My code follows the style guidelines enforced by static analysis tools.
My changes generate no new warnings.
New and existing unit tests pass locally with my changes.

Pull Request Details

Changes made:

Changed the Discussion post string to reflect the user's current timezone

To Review:

Static Analysis by Reviewer
Does it show the correct time when making a discussion post?
To check this run the following commands:
```
# From the root of Rosalution
docker system prune -a --volumes

docker compose up --build -d
```
- Go to local.rosalution.cgds/rosalution
- Login with developer
- Click a case card
- Scroll down to Discussion and click New Post
- Make a post
- Does it have the correct time you posted?
All Github Actions checks have passed.


          Updated the discussion post to show the correct time

JmScherer added the enhancement label

JmScherer self-assigned this

SeriousHorncat self-requested a review

May 6, 2024 13:20

SeriousHorncat approved these changes

View reviewed changes

Collaborator

SeriousHorncat left a comment

Screenshot says it all! Thanks James!

JmScherer merged commit a9c561a into main

8 checks passed

JmScherer deleted the fixing-discussion-display-time branch

May 6, 2024 13:59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels