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

Support custom formats for label timestamps #22

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

Conversation

lehrblogger
Copy link
Contributor

More info in the associated issue #21. I realize we could probably use the library elsewhere as well, but I wanted to minimize the changes for now.

I went back and forth a bit about whether we wanted the minified moment.js or not, but since we don't need to optimize for page load times, I figured it better for it to be readable.

This is with a format of "YYYY-MM-DD HH:mm":

screen shot 2016-11-27 at 8 01 25 pm

@lehrblogger
Copy link
Contributor Author

Oh, and I think we don't need Moment Timezone because it's handled by the environment in which Google runs the script, so that simplifies things a bit.

It is also "freely distributable under the terms of the MIT license" so I don't think we need to add/change anything in our LICENSE file.
@lehrblogger
Copy link
Contributor Author

A coworker told me about Google Apps Script Libraries today: https://developers.google.com/apps-script/guide_libraries. I think that's probably overkill for the moment.gs file, but it's still a potentially-useful thing to keep in mind.

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.

1 participant