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.

Sign up for GitHub

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

Jump to bottom

Remove non UTF-8 characters from Inner transaction Asset fields #789

Merged
merged 3 commits into from
Nov 17, 2021
Merged

Remove non UTF-8 characters from Inner transaction Asset fields #789

merged 3 commits into from
Nov 17, 2021

Conversation

algochoi
Copy link
Contributor

@algochoi algochoi commented Nov 17, 2021 •
edited
Loading

Summary

Removes non UTF-8 characters for Inner transaction asset fields.

A "proper" fix will be introduced in the 2.8.0 release.

Test Plan

Unit tests courtesy of Jason

@algochoi algochoi merged commit 57ba2b5 into hotfix/2.6.5 Nov 17, 2021
@algochoi algochoi deleted the algochoi/itxn-fields branch January 27, 2022 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jasonpaulos jasonpaulos Awaiting requested review from jasonpaulos

Assignees

@algochoi algochoi

Labels
Team Scytale Unplanned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@algochoi @onetechnical @jasonpaulos