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.

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

fix(developer): prevent invalid string ids 🏠 #12524

Conversation

mcdurdin
Copy link
Member

@mcdurdin mcdurdin commented Oct 10, 2024

Adds checks for invalid string IDs and corresponding unit tests.

Fixes: #12451
Cherry-pick-of: #12465

@keymanapp-test-bot skip

Adds checks for invalid string IDs and corresponding unit tests.

Fixes: #12451
Cherry-pick-of: #12645
@mcdurdin mcdurdin requested a review from darcywong00 as a code owner October 10, 2024 04:55
@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Oct 10, 2024

User Test Results

Test specification and instructions

User tests are not required

@keymanapp-test-bot keymanapp-test-bot bot changed the title fix(developer): prevent invalid string ids fix(developer): prevent invalid string ids 🏠 Oct 10, 2024
@keymanapp-test-bot keymanapp-test-bot bot added this to the A18S12 milestone Oct 10, 2024
@darcywong00
Copy link
Contributor

🍒 pick of #12465, right?

Copy link
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

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

lgtm

@mcdurdin mcdurdin merged commit 9511160 into stable-17.0 Oct 10, 2024
5 checks passed
@mcdurdin mcdurdin deleted the fix/developer/cherry-pick/12451-prevent-invalid-string-ids branch October 10, 2024 06:55
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 17.0.331

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants