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

Add time precision parameter to ToAtom #46

Closed
wants to merge 4 commits into from
Closed

Conversation

makkes
Copy link

@makkes makkes commented May 25, 2018

Sometimes it's desirable to have millisecond precision on atom entries.

Max Jonas Werner added 2 commits May 25, 2018 14:07
Sometimes it's desirable to have millisecond precision on atom
entries.
@kisielk
Copy link
Contributor

kisielk commented May 25, 2018

I'm open to adding this, but I don't think it should be in the form of a boolean argument. To allow for future extensibility we should probably define an AtomOptions type where there is a field for selecting the time format

@kisielk
Copy link
Contributor

kisielk commented May 25, 2018

Also, should similar functionality also be added to RSS and JSON feeds?

@makkes
Copy link
Author

makkes commented Jun 1, 2018

Yeah, I was probably just too lazy to make it an options object. :-) I'll see if I find time to refactor it. Having this feature in RSS and JSON seems reasonable, too.

@makkes
Copy link
Author

makkes commented Jul 12, 2018

I moved the precision parameter to the FeedOptions struct now. I still have to write a unit test for this.

@stale
Copy link

stale bot commented Dec 9, 2018

This issue has been automatically marked as stale because it hasn't seen a recent update. It'll be automatically closed in a few days.

@stale stale bot added the stale label Dec 9, 2018
@stale stale bot closed this Dec 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants