8000 Support decoding encodedAttributes field through the data encoder by Alex-Tideman · Pull Request #975 · temporalio/ui · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Support decoding encodedAttributes field through the data encoder #975

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 1 commit into from
Dec 6, 2022

Conversation

Alex-Tideman
Copy link
Contributor

What was changed

Add encodedAttributes field as a key to decode. Previously only payloads fields were decoded. Add new unit tests to test new field decoding.

Why?

Support decoding encodedAttributes field through the data encoder.

@vercel
Copy link
vercel bot commented Dec 6, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
holocene ✅ Ready (Inspect) Visit Preview Dec 6, 2022 at 4:36PM (UTC)
ui ✅ Ready (Inspect) Visit Preview Dec 6, 2022 at 4:36PM (UTC)

@cypress
Copy link
cypress bot commented Dec 6, 2022



Test summary

90 0 0 0


Run details

Project Temporal UI
Status Passed
Commit ccd74aa431 ℹ️
Started Dec 6, 2022 4:30 PM
Ended Dec 6, 2022 4:37 PM
Duration 06:42 💡
OS Linux Ubuntu - 22.04
Browser Chrome 107

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@Alex-Tideman Alex-Tideman merged commit d20807a into main Dec 6, 2022
@Alex-Tideman Alex-Tideman deleted the DT-185-encoded-attributes branch December 6, 2022 17:17
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
0