source-amplitude: remove date-time format from date field in average_session_length#355
Merged
Merged
Conversation
It turns out that the `average_session_length` stream's `date` field doesn't fit a `date` or `date-time` format; they're naive datetime strings like `2026-01-01T00:00:00`. Flow doesn't recognize this as a `date-time` format since there's no timezone.
These connectors have all been imported into the connectors repository, and the ATF versions have been deprecated & are no longer published. The workflows for these deprecated connectors always fail because they (correctly!) don't have permission to overwrite the images built with the imported connector. So, we may as well clean up these workflows to save a few cents.
Alex-Bair
changed the title
date format from date field in average_session_lengthdate format from date-time field in average_session_length
Alex-Bair
changed the title
date format from date-time field in average_session_lengthdate-time format from date field in average_session_length
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR's scope includes:
date-timeformat from thedatefield in theaverage_session_lengthstream. Values are naive datetime strings, which Flow does not recognize as adate-timeformat. The platform is opinionated in that regard.connectorsrepo