Skip to content

Create module for a new Workflow Trace Viewer app #1322

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

Merged
merged 2 commits into from
May 23, 2025

Conversation

wardellbagby
Copy link
Collaborator

  • Creates a new module that when ran, runs a Compose for Desktop app. The plan is for that module to host a new desktop app for viewing Workflow traces.

@wardellbagby wardellbagby requested review from zach-klippenstein and a team as code owners May 22, 2025 18:19
@CLAassistant
Copy link

CLAassistant commented May 22, 2025

CLA assistant check
All committers have signed the CLA.

@wardellbagby
Copy link
Collaborator Author

Bah, I broke Kotlin. Gimmie a bit here.

@wardellbagby wardellbagby force-pushed the wardell/create-trace-viewer-scaffold branch from e983d14 to 6f8c318 Compare May 23, 2025 03:27
@wardellbagby wardellbagby requested a review from a team as a code owner May 23, 2025 03:27
@wardellbagby wardellbagby changed the base branch from main to sedwards/update-actions-for-central May 23, 2025 03:27
@steve-the-edwards steve-the-edwards changed the base branch from sedwards/update-actions-for-central to sedwards/no-strict-trace May 23, 2025 13:47
@steve-the-edwards steve-the-edwards force-pushed the wardell/create-trace-viewer-scaffold branch from 6f8c318 to 1c9337e Compare May 23, 2025 13:52
@steve-the-edwards steve-the-edwards changed the base branch from sedwards/no-strict-trace to main May 23, 2025 14:14
wardellbagby and others added 2 commits May 23, 2025 10:15
- Creates a new module that when ran, runs a Compose for Desktop app.
The plan is for that module to host a new desktop app for viewing
Workflow traces.
@steve-the-edwards steve-the-edwards force-pushed the wardell/create-trace-viewer-scaffold branch from 1c9337e to 4529948 Compare May 23, 2025 14:16
Comment on lines +21 to +22
implementation(libs.androidx.lifecycle.viewmodel.core)
implementation(libs.androidx.lifecycle.compose)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think a lot of these are forward looking what might be useful, right? But will these androidx ones be usable?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, they're KMP now and are expected to used with Compose for Desktop.

@wardellbagby wardellbagby merged commit 64f0760 into main May 23, 2025
45 checks passed
@wardellbagby wardellbagby deleted the wardell/create-trace-viewer-scaffold branch May 23, 2025 17:37
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.

3 participants