-
Notifications
You must be signed in to change notification settings - Fork 359
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
WX-1627 Belated addition of v87 changelog entries #7436
Conversation
Co-authored-by: Adam Nichols <anichols@broadinstitute.org>
Co-authored-by: Adam Nichols <anichols@broadinstitute.org>
Co-authored-by: Adam Nichols <anichols@broadinstitute.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be a good idea to mention #7414 since it introduced a configurable field to users? Maybe something like "Added configuration option to remove 4 byte UTF-8 encoded characters (e.g. emojis) from metadata strings with the given key(s)"?
@rsaperst Excellent call-out, I did not include that here because the important bug-fix to that feature did not make it in to v87. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jgainerdewar Got it, in that case this looks good to me!
Link to rendered markdown