Skip to content
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.

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

Localizes user date #13

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Localizes user date #13

wants to merge 4 commits into from

Conversation

lukec11
Copy link

@lukec11 lukec11 commented Sep 20, 2020

sorry for the many commits — Just tested out GitHub codespaces, it's pretty cool but doesn't really match up to a local dev environment if that's available to you. In this situation I think it would've been easier to just use the GitHub web thing the whole way.

Anyway, I haven't used typescript for any significant amount of time before - if I messed something up, let me know and I will fix!

Wouldn't make sense to say "on tomorrow," "on yesterday" etc 

Will now be something like "New Journal entry from @luke | Today at 5:00PM:"
Testing this from github codespaces - pretty cool
looks like codespaces don't come with prettier installed
@rishiosaur rishiosaur closed this Sep 20, 2020
@rishiosaur rishiosaur reopened this Sep 20, 2020
@rishiosaur
Copy link
Owner

@lukec11 Can you update the imports for the rest of the files?

@rishiosaur rishiosaur self-requested a review September 20, 2020 23:36
Copy link
Owner

@rishiosaur rishiosaur left a comment

Choose a reason for hiding this comment

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

See above comments—changing the names will break the builds

@rishiosaur
Copy link
Owner

@lukec11 any progress on this?

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

Successfully merging this pull request may close these issues.

2 participants