Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

User Metadata set on start / desc #742

Merged
merged 6 commits into from
Feb 27, 2025
Merged

User Metadata set on start / desc #742

merged 6 commits into from
Feb 27, 2025

Conversation

Sushisource
Copy link
Member

What was changed

  • Added --static-summary and --static-details to all workflow-start like operations
  • Added summary & details to workflow describe

Why?

Useful feature

Checklist

  1. Closes

  2. How was this tested:
    Added tests

  3. Any docs updates needed?

@Sushisource
Copy link
Member Author

Does some but not all of #690 -- metadata query to follow in another PR

@Sushisource
Copy link
Member Author

Sushisource commented Jan 17, 2025

Actually, looks like we need to follow up on temporalio/temporal#6412 (pub-ify the fields in Go SDK) to make this work properly with schedule creation too

Comment on lines +146 to +147
StaticSummary: staticSummary,
StaticDetails: staticDetails,
Copy link
Member

Choose a reason for hiding this comment

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

I wonder if there is value putting this information in the above structured print struct, though I don't mind it being separate

@Sushisource
Copy link
Member Author

Going to wait on temporalio/sdk-go#1782 to be released (or can depend on unreleased version) so this can include schedules

@Sushisource Sushisource merged commit 728274a into main Feb 27, 2025
8 checks passed
@Sushisource Sushisource deleted the user-metadata branch February 27, 2025 18:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants