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

Passthrough start time & end time #58

Merged
merged 2 commits into from
Apr 12, 2022
Merged

Passthrough start time & end time #58

merged 2 commits into from
Apr 12, 2022

Conversation

willmarks
Copy link
Collaborator

About

With this Event API change we can now pass through the string that is provided for timestamps to the Event CLI. This now allows us to support ISO timestamps and 'Now'.

@willmarks willmarks marked this pull request as ready for review April 8, 2022 20:54
@tovbinm
Copy link
Contributor

tovbinm commented Apr 8, 2022

This now allows us to support ISO timestamps and 'Now'.

Wanna add a note about it in the README? + bump version

@willmarks
Copy link
Collaborator Author

Hmm. I realized that my Readme changes apply to both community edition and enterprise. @ypc-faros is it possible for community edition to also support all of these various forms of timestamps?

@tovbinm
Copy link
Contributor

tovbinm commented Apr 11, 2022

@willmarks let's merge. And let's create an issue here to follow up on this - https://github.com/faros-ai/faros-community-edition/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc

@willmarks willmarks merged commit 259e4bb into main Apr 12, 2022
@tovbinm tovbinm deleted the wm/timestamps branch April 13, 2022 15:26
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