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

feat: Batch export logs frontend #17605

Merged
merged 19 commits into from
Oct 6, 2023
Merged

Conversation

tomasfarias
Copy link
Contributor

@tomasfarias tomasfarias commented Sep 25, 2023

Problem

Frontend parts to display logs ingested in #17579. I mostly copied over the components used for pluginLogs and tried my best to adapt them for batch exports.

Changes

We now have tabs in batch exports UI, and logs look like this:

image

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

How did you test this code?

@tomasfarias tomasfarias force-pushed the feat/batch-exports-logs-frontend branch from ce61ea3 to 1a87007 Compare September 25, 2023 15:35
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@tomasfarias
Copy link
Contributor Author

@benjackwhite As usual, I value your input on frontend bits. But I don't want to take up much of your time, so feel free to skip this one if you are busy. Regardless, thank you!

@tomasfarias tomasfarias requested a review from a team September 26, 2023 00:30
@tomasfarias tomasfarias marked this pull request as ready for review September 26, 2023 09:05
Copy link
Contributor

@benjackwhite benjackwhite left a comment

Choose a reason for hiding this comment

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

I pushed a small style change to get rid of the duplicate header for the latest runs.
Didn't look tooo closely but didn't spot anything worth commenting on.
Nice work!

frontend/src/scenes/batch_exports/BatchExportScene.tsx Outdated Show resolved Hide resolved
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

Base automatically changed from feat/batch-exports-logs to master September 28, 2023 11:29
@tomasfarias tomasfarias force-pushed the feat/batch-exports-logs-frontend branch from 87c7655 to 469b3dc Compare September 29, 2023 08:56
Copy link
Member

@fuziontech fuziontech left a comment

Choose a reason for hiding this comment

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

Nice!

@tomasfarias tomasfarias merged commit dcf8928 into master Oct 6, 2023
72 checks passed
@tomasfarias tomasfarias deleted the feat/batch-exports-logs-frontend branch October 6, 2023 09:14
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.

5 participants